text:
vite in peerDependencies and peerDependenciesMeta causes issues in pnpm workspace

Summary
Valtio's package.json lists vite in its peerDeps. This is unnecessary, and it has the unwanted side effect that pnpm - as a documented behavior - adds vite in valtio's symlink folder name whenever vite is also present in a workspace package. The problem is that depending of this circumstance, the valtio folder name is non-deterministic, which leads to double instances, which again leads to a Symbol-related bug in valtio.
Link to reproduction
It's an intranet app, can't link it.
Check List
Please do not ask questions in issues.

[x] I've already opened a discussion before opening this issue.

Here is an exact, reproducible scenario which leads to a runtime bug:

Create a pnpm workspace with 2 packages: app and common
Make common depend on vite
But make sure that app does NOT depend on vite (app is using common's build interface, or it's using a different bundler, whatever)
Also make both app and common depend on valtio
Use valtio in app or common (it doesn't matter) like this: const store = proxyWithComputed({}, {})
Result: valtio will throw a runtime error saying that it can't determine the target as a proxy object.

The explanation of the bug is as follows:
Because valtio has peer deps which are varyingly present in app and common (let's say both are react packages, but, again, only common depends on vite), there will be 2 separate valtio folders in the pnpm store: valtio@ver+react@ver (for app) and valtio@ver+react@ver+vite@ver (for common). When there's a call to proxyWithComputed, vite resolves its import path valtio/utils to one of the folders, but then proxyWithComputed imports valtio, which vite resolves to the other folder. This leads to the effect that proxyWithComputed doesn't recognize the javascript Symbol on the proxy which valtio produced.
I couldn't determine why Vite resolves one of the import specifiers (valtio/utils) to one of the folders and the other (valtio) to the other, so there might be possibly a Vite bug here, but I concluded that in the meantime, valtio's listing of vite in its peer deps seems unreasonable, and it hijacks the pnpm behavior.
There's a workaround for the problem, by adding a .pnpmfile.cjs which cancels valtio's peerDeps entirely:
function readPackage(pkg, context) {
  if (pkg.name === 'valtio') {
    pkg.peerDependencies = {}
    pkg.peerDependenciesMeta = {}
  }

  return pkg
}

module.exports = {
  hooks: {
    readPackage
  }
}

Note that it would be enough to just delete vite from peerDeps, I just made it simpler. Also note that it's necessary to fix both peerDependencies and peerDependenciesMeta.

I think we can remove vite from peerDependencies/Meta. Can you open a PR?
We might need to add an exception rule for eslint.
@Aslemammad Does that sound okay?

I think it is okay to remove it if we let the user know that they should install vite.

@Aslemammad It doesn't make sense to me. If valtio depends on vite, then it should be listed among its deps (peer or not). But why would valtio depend on vite at all??

It's because of the macro plugin, it depends on vite! But we do not want users to download Vite with valtio. So we either have it in the peers or we remove it.

@Aslemammad I made a PR which removes all the unnecessary peer deps. React should be the only one. I'm still confused by your earlier remark saying that "let the user know that they should install vite". All in all, it seems to me that all those cleaned up peer deps should belong to dev deps, and they were actually listed under dev deps too. This in itself seems contradictory (deps listed in multiple places), as peer deps mean "shared", other deps mean "not shared", the 2 together doesn't make sense. React is also listed under dev deps, which to me seems like a mistake - package managers probably ignore the devDeps/react clause, as it's already in peer, rightfully - it should be shared with the consuming package; but I didn't want to overextend with this PR, I was only concerned with fixing my bug :)

The remark means that we document it.
And around listing deps into multiple places, one note is that peer deps won't get installed, so for development of valtio, we need those packages. So we need also to put them into the dev deps.

But why should the user install vite in order to use valtio? And in the next comment you're saying "But we do not want users to download Vite with valtio." So which one?

If it's in the devDependencies, it won't get downloaded. if it's in the peers, the user would get notified or something like that. what we want, is that if the user wants to use the macro plugin, they should install vite, but if they do not use it, then no need to install.

And how is that macro plugin related to valtio?

https://github.com/pmndrs/valtio/tree/main/src/macro

Is there any docs on what this is, or can you brief me? I can't figure out, I'm not familiar with macros.

Sure, here you go: link

I see, thanks. I see the whole reasoning of the peer dep config now, and it makes sense. I guess the problem is that some opt-in feature has different dep requirements than the main package. (Even react is an opt-in feature.) And the question is whether to add their dependencies to the package or not. I guess it's a matter of principle, one can follow either way. The typical solution would be to split those features out into separate npm packages, and have the peer dep settings in them, but I wouldn't insist on it.
As for the build tools (vite, babel macro stuff), they still look iffy... I would remove them with a peace of mind. Peer deps are not just "vs nothing" documentational stuff which says "hey you should have these installed as well", but also "vs double instance", which tells your pkg manager to have at most 1 instance from these, to avoid double runtime instances. Double react instances were a common issue at a time when lib authors sometimes just threw it under regular deps. I think this is the main use case of peer deps, otherwise you could just add your deps into regular deps instead of telling the user to install them. Now this aspect is completely irrelevant for the build tools, that's why I say I would be easy to remove them from peer deps.
React itself is an interesting question though. As react is also an opt-in feature of valtio, it can still happen that a pnpm workspace has e.g. a vue and a react app, both using valtio, at it'll break. Also, if a standalone vue app uses valtio in an npm project, they'll have react installed unnecessarily, as npm auto installs peer deps. (Btw, I don't know why pnpm doesn't do it by default, I configured it to do so in my workspaces.) It's not a big deal, again, more of a principle matter.
But react is such a "commonplace peer dep", and maybe we can say a major use case environment for valtio, that it's hard to suggest not to have it in there. So it's up to you guys. Technically the only benefit is that someone wants to use the hooks without having react installed, they'll be covered. The risk is this issue. In any way, my current PR covers my suggestion: leave react in, move build stuff out.
I also remind you guys, that this might be a vite glitch as well. In my debugging, I've found that proxyWithComputed imports vanilla from the other valtio instance, which shouldn't happen, as this line makes it unequivocal that it imports it from  its own parent folder. But it might make sense in vite's logic in some way. It really goes down to the internals of how vite resolves the '../vanilla' specifier to 'valtio', and then how 'valtio' is symlinked to one package, and 'valtio/utils' to the other by pnpm. Hard to tell. I didn't have capacity to trace it down, as I wasn't exactly sure that vite or pnpm is faulty. My focus was that seeing a double valtio instance in the pnpm store is smelly when I have valtio as peer in my common pkg. Even if I'm not 100% sure that a double valtio instance shouldn't work (because why not?), it just looked iffy.
Sorry for my wordy style. Bottom line is, my PR stands as it is.

Yea, sure. At the end, it's about @dai-shi if he thinks the PR is proper. Thank you for the work

authors: Aslemammad dai-shi sarimarton

repo: pmndrs/valtio


text:
fix(utils): re-implement derive to make glitch free (#335 was buggy)

Related: https://github.com/pmndrs/valtio/issues/296#issuecomment-1020736885

[x] add failing test @Noitidart
[x] re-implement @dai-shi
[x] refactor @dai-shi

Okay, I kind of new my impl in #335 was truly correct. I had another impl, and based on it, a commit is added. It's complicated, so let me try some refactoring.
Meanwhile, @Noitidart could you try if it solves the issue?

Let's release this soon, because #342 is a bigger issue. v1.2.10 is full of bugs. 😓

Okay, I kind of new my impl in #335 was truly correct. I had another impl, and based on it, a commit is added. It's complicated, so let me try some refactoring.
Meanwhile, @Noitidart could you try if it solves the issue?

Trying now! Sorry for delay I'm on west coast us so just woke up haha.

With v1.2.9 this used to happen:
compute1
compute2
compute3:
000
render value:
v0: 0, v1: 0, v2: 0
INCREMENT
compute1
compute3:
110
compute2
compute3:
111
render value:
v0: 1, v1: 1, v2: 1

Notice how compute1 then compute3 happens, then again compute2 happens then compute3 happens.
Now in v1.2.11 its perfect:
compute1
compute2
compute3:
000
render value:
v0: 0, v1: 0, v2: 0
INCREMENT
compute1
compute2
compute3:
111
render value:
v0: 1, v1: 1, v2: 1

compute1 then compuet2 then compute3 and all variables are lined up! no extra callback is triggered!

Trying now! Sorry for delay I'm on west coast us so just woke up haha.

No worries! I knew the timezone difference. I wanted to replace troublesome v1.2.10 as soon as possible.

Totally gotcha, thanks for giving me chance to take part of the emergency release. If possible it happens again no matter time if I see it I would definitely get on it, thanks for trying to include us man!

By the way, this implementation might still have a limitation. A circular graph may result in an infinite loop. Would you like to test it??
(It may happen with v1.2.9 too.)

By the way, this implementation might still have a limitation. A circular graph may result in an infinite loop. Would you like to test it?? (It may happen with v1.2.9 too.)

Haha thanks! I don't understand the circular graph situation, if you could help me understand I would learn about this circular graph thing.

The circular graph is something like this: A derived from B derived from C derived from A.

Found this: https://en.wikipedia.org/wiki/Reactive_programming#Cyclic_dependencies
It's called cyclic dependencies.

Oh thanks! Sorry I got busy with urgently implementing a Stripe payment system update. I'll have to revisit this please. Thanks very much for trying to get me involved!

source: gharchive/pull-request

authors: Noitidart dai-shi

license: MIT

license_type: permissive

license_source: github-api


text:
The example in the PDF uses an internal URL

The example should be moved to a regular Bluemix URL

After you push the sample as one bluemix app, and bind one PA service instance to it, the URL will be obtained from OS env automatically.

source: gharchive/issue

authors: data-henrik rodney-vin

repo: pmservice/predictive-modeling-samples

license_source: bigquery


text:
분류모델 - PosterV2

감정 분류 모델 - PosterV2 (github)
[ ] pretrained weights 가져와서 우리 샘플 데이터셋(0.005)으로 테스트 해보기
- raf-db-model_best.pth 사용 top1 avg accuracy 19.513

[ ] 우리 데이터로 학습 후 평가

노트북에 코드 구조 정리 (colab)


text:
Docs: Remove Get-TeamHelp

Since v1.0.6, this command is removed/deprecated. No need to show this in the comparison list.

Good catch. Thank you! 👍

This is a quick fix. Just put it on my name 👍


text:
Enhancement: Rename and add alias to spo list label set and spo list label get

I find the command names for the following commands a bit unclear:
spo list label set
https://pnp.github.io/cli-microsoft365/cmd/spo/list/list-label-set/
spo list label get
https://pnp.github.io/cli-microsoft365/cmd/spo/list/list-label-get/
It's not clear what kind of label this is about. It would be clearer if we would use the word retentionlabel here. This seems to be used in purview these days instead of the older 'compliance tag'.
Renaming the command would be a breaking change. I'd suggest renaming them AND adding the current one as alias.
The new names should be:
spo list retentionlabel get
spo list retentionlabel set
Even if it's longer, it's clearer in my eyes.

fine for me 👍
@pnp/cli-for-microsoft-365-maintainers any other comments ?

I am sorry, I can't comment on naming convention since I have not worked much in that area.
Having said that, your reasoning seems logical to me too.

What other types of labels are available in SharePoint that make you feel this name is ambiguous?

Well, Sensitivity labels for example.
https://learn.microsoft.com/en-us/microsoft-365/compliance/sensitivity-labels?view=o365-worldwide
But aside from that, label is an ambiguous term. Even regular metadata could be perceived as labels you add to an item.
Or you could perceive the word as a categorization mechanism.
You simply do not know what's meant immediately. Retention label communicates its intent much better.

By the way, I've updated the specs as I'd mistakenly written spo label set instead of spo list label set
And spo retentionlabel set instead of spo list retentionlabel set

Well, Sensitivity labels for example.

From what I read, sensitivity labels are stored in files, not set on a list, so I wonder if the ambiguity truly applies here, given our context is spo list, not Microsoft 365 all-up.

Yes, but then again: as a user you might not be familiar with the different labels and their application, which makes it easy to get confused. I myself was confused last week, when searching for cli support of retentionlabels.
It still is a generic term.
Also: since we're doing some work on purview retentionlabels, it makes sense to name the commands the same across the board. Those commands include applying retentionlabels to files as well, which means naming every occurrence label is not going to work.

Understood. As we're about to release v6, we should:

rename the command to use the new name
add the alias using the old (current) name to avoid breaking changes
add a deprecation warning of the old name (alias)
add a new issue linked to the v7 milestone to remove the deprecated alias and update the v7 upgrade guidance

authors: Adam-it arjunumenon martinlingstuyl waldekmastykarz


text:
Update 'approleassignment add' to v1.0 API. Closes #1858

Update approleassignment add to v1.0 API. Closes #1858

Awesome @aakashbhardwaj619. We'll have a look at it asap 👏

Merged manually. Thank you! 👏


text:
Fixes user-agent header. Closes #1951

Fixes user-agent header. Closes #1951

Merged manually. Thank you 👍

authors: appieschot waldekmastykarz


text:
Extend spo list webhook with option listUrl, closes #3812

Closes #3812

Thank you @MathijsVerbeeck
We'll review it ASAP!

Wow @MathijsVerbeeck lightning fast, awesome! 😀
I'll have a 2nd look ASAP!

Merged manually, thank you!

authors: MathijsVerbeeck milanholemans


text:
Pages visible, but the communication site is still the default blank template

I setup the app catalog and manually uploaded v3.0.1 to the catalog in SPO. I was able to run the script to complete the provisioning by modifying a line to work with MFA "Connect-PnPOnline -Url $clSite -SPOManagementShell".
The CustomLearningViewer and CustomLearningAdmin pages are visible and load as well. My issue is that the home page of the communication site isn't customized like all of the screenshots I've seen. I also don't see links in the navigation for Microsoft 365 training and asking a question. Are those all custom configurations and everyone is just using those screenshots as examples?

That's because you only get those custom pages when you provision learning pathways using the provisioning service, not when you do a manual provision.

Thanks Julie, I'll try to provision another site using that. I had already tried to use the provisioning service a bunch of times, but it kept failing.
I was hoping to avoid doing all of the customization on my own, since I like the look of the templates and want to use that as a baseline.

I don't know if the provisioning service supports MFA. Further, you can only have one learning pathways administrative site in a tenant, so if you provision learning pathways to a different site you will end up abandoning your original site. Further, I'm not sure how nice the provisioning service plays when you already have the app in the app catalog that in this case would be an older version.  The new version only addresses upgrade scenarios from v2 so it doesn't really matter which version you have but you should be aware.

I just retried the provisioning and it worked this time! I didn't get rid of my manual site beforehand, since I had started provisioning before I saw your comment. The only difference as far as I know is that when trying Connect-PnPOnline -PnPO365ManagementShell with the manual script it asked me to accept additional permissions to our tenant prior to failing.
Either way I'll close this out as it worked fine with the manual process per your comments.
Thanks again


text:
What is the difference from version 4.1.3 and 4.2?

Submit your question
What is the context of your question

[ ] Learning Pathways Content

What is your question

What is the difference from version 4.1.3 and 4.2? and is there difference in content?

Hi @FadyJaradat
Content Updates and Version Updates occur separately and details for each can be viewed at the below locations.
Content Updates
Version Updates

Closing as no response has been received for 30 days past the given information


text:
Added a new sample Power365CRM

We're so eager to add your sample to this repository as soon as possible, but we need all samples follow the same structure.
To make sure we process it as soon as possible, make sure to follow these instructions.

Q
A

Bug fix?
no

New feature?
yes

New sample?
yes

Related issues?
none

Solution Details

Q
A

Solution requires premium license
no

Solution uses experimental features (see licensing guidelines)
no

Which connectors are used?
DataVerse

Solution uses on-premises connectors?
no

Solution uses custom connectors?
no

Layout
Tablet

What's in this Pull Request?

Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
Power 365 CRM is a community solution based on Microsoft Dataverse low-code and pro-code. In this release v1.0 you will be able to manage your entire sales process from Prosect to Quote. This will include management of opportunities during the sales process and autonomous opportunity amount calculation. This Pull request contains two solutions which contains a model driven app, multiple tables, 2 custom pages, 3 custom APIs, 3 plugins and some web resources.
Note: for security reasons, we can only accept pull requests which consist of unpacked source code for .msapp files; We cannot accept .msapp files.

Hi @Laskewitz, this is Wilmer I was wondering if anyone in the team has been able to see the pull request.

repo: pnp/powerapps-samples


text:
Update README.md

ouch.
I dont want my tenant id out in the wild!

Sorry I didn't spot that in the review. I have accepted urgently, site will auto update shortly.

authors: pkbullock russgove

repo: pnp/script-samples


text:
People Picker is not Cleared on button submit in SPFX Tab Control

People picker is cleared when changing the tabs or refreshing the page ...but it is not cleared on clicking the submit button... i used [] to clear ...but its not worked

Hi @Binz95,
Thanks for reporting this!
I believe that "Submit" button just runs some JS code, with no actual submit or refresh.
In that case PeoplePicker will remain its current state.
Unfortunately, there is no direct way to "clear" the selection. We will need to add this feature at some point.
You can try 2 things:

when clicking on Submit you can remove and re-add the control.
you can set defaultSelectedUsers and change one of the next properties: groupName, webAbsoluteUrl, principalTypes. It should lead to refreshing the state.

Let me know if you have any additional questions.

Can you please give me a demo or example to remove /add the controller
On Wed, 10 Feb 2021, 6:36 am Alex Terentiev, notifications@github.com
wrote:

Hi @Binz95 https://github.com/Binz95,
Thanks for reporting this!
I believe that "Submit" button just runs some JS code, with no actual
submit or refresh.
In that case PeoplePicker will remain its current state.
Unfortunately, there is no direct way to "clear" the selection. We will
need to add this feature at some point.
You can try 2 things:

when clicking on Submit you can remove and re-add the control.
you can set defaultSelectedUsers and change one of the next
properties: groupName, webAbsoluteUrl, principalTypes. It should lead
to refreshing the state.

Let me know if you have any additional questions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pnp/sp-dev-fx-controls-react/issues/777#issuecomment-776355794,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKZIFAKNWUN4QWW7EYBYQCTS6HLYPANCNFSM4VS45XDA
.

export interface IPeoplePickerClearState {
  selectedPeople?: string[];
  reset?: boolean;
}

export default class PeoplePickerClear extends React.Component<IPeoplePickerClearProps, IPeoplePickerClearState> {

  constructor(props: IPeoplePickerClearProps) {
    super(props);

    this.state = {};
  }

  public render(): React.ReactElement<IPeoplePickerClearProps> {
    return (
      <div className={styles.peoplePickerClear}>
        <PeoplePicker
          context={this.props.context}
          onChange={(newItems) => this.setState({
            selectedPeople: newItems.map(i => i.text)
          })}
          defaultSelectedUsers={this.state.selectedPeople || []}
          webAbsoluteUrl={this.state.reset ? this.props.context.pageContext.web.absoluteUrl : undefined}
        />
        <PrimaryButton text={'Reset'} onClick={() => {
          this.setState({
            reset: true,
            selectedPeople: []
          });

          setTimeout(() => {
            this.setState({
              reset: false
            });
          }, 50);
        }} />
      </div>
    );
  }
}

Thanks, I'll check it out.
On Sat, 13 Feb 2021, 4:21 am Alex Terentiev, notifications@github.com
wrote:

export interface IPeoplePickerClearState {
selectedPeople?: string[];
reset?: boolean;}
export default class PeoplePickerClear extends React.Component<IPeoplePickerClearProps, IPeoplePickerClearState> {
constructor(props: IPeoplePickerClearProps) {
super(props);
this.state = {};

}
public render(): React.ReactElement {
return (

<PeoplePicker
context={this.props.context}
onChange={(newItems) => this.setState({
selectedPeople: newItems.map(i => i.text)
})}
defaultSelectedUsers={this.state.selectedPeople || []}
webAbsoluteUrl={this.state.reset ? this.props.context.pageContext.web.absoluteUrl : undefined}
/>
<PrimaryButton text={'Reset'} onClick={() => {
this.setState({
reset: true,
selectedPeople: []
});
      setTimeout(() => {
        this.setState({
          reset: false
        });
      }, 50);
    }} />
  </div>
);

}}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pnp/sp-dev-fx-controls-react/issues/777#issuecomment-778499105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKZIFANQETEMLEDTIJL7DZDS6WWGPANCNFSM4VS45XDA
.

Closing due to no activity.

repo: pnp/sp-dev-fx-controls-react


text:
Fix TextWithCallout read-only in SPFx 1.12.1

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

Related issues?
NA

What's in this Pull Request?
After SPFx 1.12.1 update, looks like something changed in Fabric UI and that causes TextFieldWithCallout to be read-only. This PR fixes that

Thank you @gautamdsheth for the fix!

authors: AJIXuMuK gautamdsheth

repo: pnp/sp-dev-fx-property-controls


text:
Unable to run example code in Usage

I can't seem to get the example code working from your Usage section. Code:
`import time
import timeout_decorator
@timeout_decorator.timeout(5)
def mytest():
print "Start"
for i in range(1,10):
time.sleep(1)
print "%d seconds have passed" % i
if name == 'main':
mytest()
`
Error:
me$ python2.7 tod.py  Start 1 seconds have passed 2 seconds have passed 3 seconds have passed 4 seconds have passed Traceback (most recent call last):   File "tod.py", line 13, in <module>     mytest()   File "/Users/me/anaconda2/lib/python2.7/site-packages/timeout_decorator/timeout_decorator.py", line 69, in new_function     return function(*args, **kwargs)   File "tod.py", line 9, in mytest     time.sleep(1)   File "/Users/me/anaconda2/lib/python2.7/site-packages/timeout_decorator/timeout_decorator.py", line 60, in handler     raise TimeoutError() timeout_decorator.timeout_decorator.TimeoutError: u'Timed Out'

looks like it works based on your output

repo: pnpnpn/timeout-decorator

license: mit


text:
(Fix) Fix gas price validation

Fixes #645.

Pull Request Test Coverage Report for Build 1544

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 9.49%

Totals

Change from base Build 1538:
0.0%

Covered Lines:
257

Relevant Lines:
1954

💛  - Coveralls

authors: coveralls fvictorio

repo: poanetwork/token-wizard


text:
I want re-install dock widget

I had issue about dock widget update. When I try to updating dock widget, Pock is frozen.
So, I deleted dock widget. But I can't re-install dock widget. I have to know download link for dock widget.
Can I get to know link?
I installed pock again. But dock widget was equally gone.

Same issue here, pock perpetually tells me to update the dock widget, it will never update. Have reinstalled pock many times. Package always fails to install.

#455

authors: WristyManchego YounggyuKim rudzainy

repo: pock/pock


text:
[bug] permissions for single related fields

Hello. I found an odd behavior on PocketBase v0.23.4 and JS client 0.22.0
I have two tables: portfolios and portfoliosMeta. portfoliosMeta has a single relation to portfolios and has name portfolio
To protect records from other users I have these permission rules
@request.auth.id = @collection.portfolios.user.id && @collection.portfolios.id = @collection.portfoliosMeta.portfolio.id

// also I've tried a simplified version
@request.auth.id = @collection.portfolios.user.id && @collection.portfolios.id = portfolio

and it works correctly until I add more then one portfolios - portfoliosMeta relation, and after that it drops 404
I call it like this from the client:
pb
    .collection(Collections.PortfoliosMeta)
    .getFirstListItem(`portfolio = '${portfolioId}'`, {
      expand: "portfolio",
    })

Also, I tried different get methods from the pb js client.
Maybe it works correctly and I didn't understand the permission rules, but anyway thanks for the help in advance
I'm not sure when it happened, but as additional information, I have one more table with the same single relation to portfolios

It is not a bug, or at least it is not is not clear how to reproduce it.
It is also not clear for which collection is supposed the above rule to apply.
@collection.* and other fields that results in multiple value apply by default a "match-all" constraint. If you want "any/at-least-one" type of constraint you need to prefix the operator with ?, for example:
@request.auth.id ?= @collection.portfolios.user.id && @collection.portfolios.id ?= portfolio

As a general recommendation, avoid excessive use of @collection.* fields as it could degrade the performance of your application if not use with proper care. If all you want is to filter a specific field of a relation, then using direct dot-notation should be sufficient:
@request.auth.id != "" &&
@request.auth.id = portfolio.user

If you have further questions, please use the Q&A discussions and paste an export of your collections configuration as json (Admin UI > Settings > Export collections).

authors: ganigeorgiev mirus-ua

repo: pocketbase/pocketbase


text:
Fixed Two Bugs on Dashboard

Sidebar Nav Item Not Staying highlighted when not on overview.
Overview Information not updating when switching between instances.

🔥 thank you so much!

authors: LuminareHQ benallfree

repo: pockethost/pockethost


text:
[Feature] Create campaign types filter for campaigns page

[x] Created filter for campaign types for campaigns page based on #613
[x] Changed styles and text for heading and subheading
[x] Removed apply button

https://user-images.githubusercontent.com/92337248/158852120-b41cce2d-45e9-43e0-81fb-0574867ce3b9.mp4

@Yakimov1337 The backend for this task is extracted in https://github.com/podkrepi-bg/api/issues/146

Desktop
Mobile

repo: podkrepi-bg/frontend


text:
body.setting.resourceTimeout causing ajax and script references error

At times, script references are not loaded and ajax requests return http
status 0 with body.settings.resourceTimeout 1000.
The idea is that this timeout is configurable?
https://github.com/walter-psjr/phantom-html-to-pdf/commit/7dde10d6204d015d8260a771a305fac9c4082f13

the resource timeout is already configurable
var conversion = require("phantom-html-to-pdf")();

conversion({
  // other options...
  settings: {
    resourceTimeout : 180000
  }
// other options..
}, function(err, pdf) {
});

authors: bjrmatos walter-psjr

repo: pofider/phantom-html-to-pdf


text:
Change font type

Hi,
Actually, when i generate PDF with your library i get trueType (CID) Font type in my document (See image below) but i don't want this kind of font type.
How i can change this behaviour ?
Thanks for your help

This library is just a wrapper around phantomjs.
Please try to check phantomjs issues directly.
https://github.com/ariya/phantomjs

authors: NTurchi pofider


text:
.runtimeWarning reporter warnings appear at incorrect source location if caller provides an error

If you call reportIssue("foo") you will see a runtime warning on the correct line because _RuntimeWarningReporter overrides reportIssue(_:) with @_transparent.
However, if you call reportIssue(MyError(), "foo") the purple warning will instead show up at IssueReporter.swift#L88 because _RuntimeWarningReporter utilizes the default implementation of the method which doesn't use the @_transparent attribute.
Version 1.4.2, Xcode 16.0, iOS simulator

Hi @Killectro, yeah this is just an oversight. Would you be willing to PR a fix?

authors: Killectro mbrandonw

repo: pointfreeco/swift-issue-reporting


text:
scl provider on RHEL inconsistencies

Hi there,
So I'm having some real problems getting a consistent python 2.7 install on RHEL 6.6.
This is how I'm installing it in my recipe:
python_runtime 'vagrant' do
  provider :scl
  version '2.7'
end

Most of the time it fails with this error during provisioning (I'm using the cookbook with vagrant+chef):
==> vagrant-slave-2: Mixlib::ShellOut::ShellCommandFailed
==> vagrant-slave-2: ------------------------------------
==> vagrant-slave-2: rpm_package[rhscl-python27] (/var/chef/cache/cookbooks/poise-languages/files/halite_gem/poise_languages/scl/resource.rb line 112) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
==> vagrant-slave-2:
==> vagrant-slave-2: ---- Begin output of rpm -qp --queryformat '%{NAME} %{VERSION}-%{RELEASE}
==> vagrant-slave-2:
==> vagrant-slave-2: ' https://www.softwarecollections.org/en/scls/rhscl/python27/epel-6-x86_64/download/rhscl-python27-epel-6-x86_64.noarch.rpm ----
==> vagrant-slave-2:
==> vagrant-slave-2: STDOUT:
==> vagrant-slave-2:
==> vagrant-slave-2: STDERR: curl: (7) couldn't connect to host
==> vagrant-slave-2:
==> vagrant-slave-2: error: open of https://www.softwarecollections.org/en/scls/rhscl/python27/epel-6-x86_64/download/rhscl-python27-epel-6-x86_64.noarch.rpm failed: No such file or directory
==> vagrant-slave-2:
==> vagrant-slave-2: ---- End output of rpm -qp --queryformat '%{NAME} %{VERSION}-%{RELEASE}
==> vagrant-slave-2:
==> vagrant-slave-2: ' https://www.softwarecollections.org/en/scls/rhscl/python27/epel-6-x86_64/download/rhscl-python27-epel-6-x86_64.noarch.rpm ----
==> vagrant-slave-2:
==> vagrant-slave-2: Ran rpm -qp --queryformat '%{NAME} %{VERSION}-%{RELEASE}
==> vagrant-slave-2:
==> vagrant-slave-2: ' https://www.softwarecollections.org/en/scls/rhscl/python27/epel-6-x86_64/download/rhscl-python27-epel-6-x86_64.noarch.rpm returned 1

I'm not sure exactly what it's doing here? It looks like it's running an rpm command which results in a curl call? And this curl call is failing (Trying it manually results in an empty file?).
However if I run the exact same chef cookbook again a 2nd time, it goes through fine and seems to install python from the repository:
==> vagrant-slave-2: [2015-11-13T12:57:32-05:00] INFO: rpm_package[rhscl-python27] installed rhscl-python27 at 1-2
==> vagrant-slave-2: [2015-11-13T12:57:32-05:00] INFO: ruby_block[flush_yum_cache] called
==> vagrant-slave-2: [2015-11-13T12:57:35-05:00] INFO: yum_package[python27] installing python27-1.1-17.el6 from rhscl-python27-epel-6-x86_64 repository
==> vagrant-slave-2: [2015-11-13T12:58:31-05:00] INFO: yum_package[python27] installed python27 at 1.1-17.el6

I've been trying to troubleshoot the first failure and I have no idea what's causing it, or why the 2nd run would work fine. Any ideas?
Thanks!

Ok I think I'm seeing the issue... I'm using a proxy on my end. It looks like whatever shell is spun off to run the following command is not taking into account proxy variables?
rpm -qp --queryformat '%{NAME} %{VERSION}-%{RELEASE}
' https://www.softwarecollections.org/en/scls/rhscl/python27/epel-6-x86_64/download/rhscl-python27-epel-6-x86_64.noarch.rpm

Is it possible the shell that is used to run this doesn't properly inherit OS environment variables? If I unset https_proxy I can reproduce the same error.
$ unset https_proxy
$ rpm -qp --queryformat '%{NAME} %{VERSION}-%{RELEASE}
' https://www.softwarecollections.org/en/scls/rhscl/python27/epel-6-x86_64/download/rhscl-python27-epel-6-x86_64.noarch.rpm
curl: (7) couldn't connect to host
error: open of https://www.softwarecollections.org/en/scls/rhscl/python27/epel-6-x86_64/download/rhscl-python27-epel-6-x86_64.noarch.rpm failed: No such file or directory

I'm guessing this isn't actually needed and the 2nd run keeps going past it and completes successfully?

Guessing between the first and second rounds something gets written out that makes RPM work with the proxy? Can you try setting ENV['https_proxy'] = 'whatever' in the code before you hit the python install?

Yeah that could be it - proxy gets loaded via a file provisioned in /etc/profile.d/, maybe it doesn't get sourced until the 2nd time around. Didn't get to the bottom of it as I needed to build from source so I'm back to the origin python cookbook. Thanks for the quick reply.

authors: coderanger jfbibeau

repo: poise/poise-python


text:
Add two more selections

Add selections located at:

https://kriper.net/golden-foundation/
https://kriper.net/top.html

golden foundation stoies selection: done

Done

repo: pokatomnik/kriper


text:
feat(dex): Sort Pokemon by Generation

Adds a button to sort the Pokemon in a dex by Generation as well as Dex Number.
Fixes: #420

I've been trying to start my living dex from the beginning and I want each generation of Pokemon to be caught in the generation they were introduced, however this is a bit of a problem since the earliest games supported are Gen 6. I was using the ORAS National Dex for Emerald but it made figuring out where to put which Pokemon in my boxes a bit more difficult since I had to skip over the older Gens and the newer Gens (such as Gen 4 evolutions/babies of Gen 3 Pokemon). This functionality makes it a lot easier to use the newer dex definitions to track for older games.
I think with this all you need is a dex type that returns all Pokemon for all games and you'd have a nice way to track your entire living dex in one place, rather than splitting them up into multiple dexes

hey @Garethp!
our preferred way to solve this problem is to actually add support for the older games (including capture info), as opposed to adding sorting. we're not trying to optimize for using the current gen dexes to track older games.
thanks anyway for the contribution! we'll be cleaning up our submitted issues so it can be more clear which things are actionable or not.

I still believe that sorting by generation has it's own use-case, especially if you want to have each generation in it's own box (for example, in Home I'd rather have my Gen 2 Pokemon in a separate box from my Gen 3 Pokemon. Being able to see that Treeko is the first Pokemon in it's own box rather than at the end of row 2 of a box that starts with Miltank is useful to me. I personally end up running my forked webapp locally and have it talk to your API for this exact reason.
That being said, I'm aware this is an open source project and for the issue of supporting older games you'd rather add support directly such as adding capture info. How can I contribute that data? It doesn't look as if the full data set is in any of the Github  repos. If possible, I'd like to help contribute to extending this project's support for games and features in ways that you prefer.

authors: Garethp ami


text:
Consider using PureJSONSwift

see https://gist.github.com/fabianfett/efb4bc16e402e632eda9af9e8e2a3972
The only place JSON encoder is used is here:
https://github.com/pokryfka/aws-xray-sdk-swift/blob/master/Sources/AWSXRayRecorder/Segment%2BJSON.swift
which is (or should?) used only by UDPEmitter
also LogEmitter creates an instance with "pretty" formatting - it could be moved to XRayTesting

Consider JSONLogHandler for LogEmitter (?)

Yeah it would be great to avoid the foundation one, for a number of reasons: 1) perf and 2) just the foundation dependency.
Ping @fabianfett since we talked a while ago about xray and purejson :)

@ktoso We are way ahead!

@ktoso @fabianfett is there a good explanation why the JSON encoder from Foundation is so slow on Linux (performance is similar on macOS)

Result of CodingPerfTests
swift:5.2-amazonlinux2 running in Docker with 2 CPUs and 4 GiB memory on macOS 10.15.6 with 3 GHz 6-Core Intel Core i5, 32 GB:
Number of invocations: 10000
------------------------------------------
JSONValue to bytes
PureSwift                     | took: 0.54722s
------------------------------------------
Encoding
Foundation                    | took: 12.13563s
Ikiga                         | took: 0.38421s
PureSwift                     | took: 1.17607s
------------------------------------------
Reading
PureSwift on [UInt8]          | took: -0.00000s
PureSwift on Foundation.Data  | took: 0.69957s
PureSwift on NIO.ByteBuffer   | took: 2.38591s
------------------------------------------
Parsing
Foundation on Foundation.Data | took: 2.03149s
PureSwift on [UInt8]          | took: 0.50969s
PureSwift on Foundation.Data  | took: 0.50999s
PureSwift on NIO.ByteBuffer   | took: 0.53209s
------------------------------------------
Decoding
Foundation on Foundation.Data | took: 9.99868s
Foundation on NIO.ByteBuffer  | took: 9.98646s
IkigaJSON on [UInt8]          | took: 2.49468s
IkigaJSON on Foundation.Data  | took: 2.50112s
IkigaJSON on NIO.ByteBuffer   | took: 2.50956s
PureSwift on [UInt8]          | took: 1.07665s
PureSwift on Foundation.Data  | took: 1.08228s
PureSwift on NIO.ByteBuffer   | took: 1.09722s

for reference macOS 10.15.6, Swift 5.2.4 on 3 GHz 6-Core Intel Core i5, 32 GB:
Number of invocations: 10000
------------------------------------------
JSONValue to bytes
PureSwift                     | took: 0.46937s
------------------------------------------
Encoding
Foundation                    | took: 2.71739s
Ikiga                         | took: 0.50643s
PureSwift                     | took: 1.48155s
------------------------------------------
Reading
PureSwift on [UInt8]          | took: 0.00000s
PureSwift on Foundation.Data  | took: 0.36042s
PureSwift on NIO.ByteBuffer   | took: 2.37401s
------------------------------------------
Parsing
Foundation on Foundation.Data | took: 0.46644s
PureSwift on [UInt8]          | took: 0.64662s
PureSwift on Foundation.Data  | took: 0.65942s
PureSwift on NIO.ByteBuffer   | took: 0.66094s
SwiftyJSON                    | took: 1.99765s
------------------------------------------
Decoding
Foundation on Foundation.Data | took: 3.09398s
Foundation on NIO.ByteBuffer  | took: 3.11534s
IkigaJSON on [UInt8]          | took: 3.06283s
IkigaJSON on Foundation.Data  | took: 3.06483s
IkigaJSON on NIO.ByteBuffer   | took: 3.06710s
PureSwift on [UInt8]          | took: 1.61549s
PureSwift on Foundation.Data  | took: 1.61314s
PureSwift on NIO.ByteBuffer   | took: 1.61946s

Takeaways:

Foundation JSON encoder has terrible performance on Linux
PureSwift JSON encoder is ~10 time faster than Foundation JSON encoder
IkigaJSON encoder is ~30 time faster than Foundation JSON encoder  (!)

PureSwift JSON encoder returns an array of bytes:
public struct PSJSONEncoder {

    public var userInfo: [CodingUserInfoKey : Any]

    public init()

    public func encode<T>(_ value: T) throws -> [UInt8] where T : Encodable

    public func encodeAsJSONValue<T>(_ value: T) throws -> PureSwiftJSON.JSONValue where T : Encodable
}

IkigaJSON JSON encoder uses Foundation.Data
public struct IkigaJSONEncoder {

    public var userInfo: [CodingUserInfoKey : Any]

    /// These settings influence the encoding process.
    public var settings: IkigaJSON.JSONEncoderSettings

    public init()

    public func encode<E>(_ value: E) throws -> Data where E : Encodable
}

(Average) results in seconds of PerformanceTests in https://github.com/pokryfka/aws-xray-sdk-swift/pull/42 from check_run_id=934347557

Test
test-macos (ubuntu-latest)
test-linux (swift:5.2)
test-linux (swiftlang/swift:nightly-5.3-bionic)

testEncodingUsingFoundationJSON
0.692
0.417
0.413

testEncodingUsingIkigaJSON
0.564
0.062
0.076

testEncodingUsingPureSwiftJSON
0.750
0.079
0.097

Note relative standard deviation is ~40% on macos and ~3% on linux (?).

Did some testing, key takeaways:

Foundation JSON encoder has terrible performance on Linux
IkigaJSON JSON encoder is consistently faster than PureSwift JSON encoder
IkigaJSON has dependency on Foundation.Data

@fabianfett @ktoso
Using Foundation JSON encoder does not make sense in library targeted for Linux because of its much worse performance.
However I am not sure how much the dependency on Foundation.Data is an issue.
For example I did not test if, and by how much, it would reduce the binary size, loading time and so.

Using Foundation JSON encoder does not make sense in library targeted for Linux because of its much worse performance.

Yep. It's something we're painfully aware of;
I don't have a concrete plan yet about what we'll do there but yes it's a huge pain point for the server ecosystem and we'll need to improve the situation here. Status quo basically is: avoid Foundation (in general) on server if possible, and definitely avoid Foundation's coders if you can by depending on Pure or Ikiga.

PureSwift JSON encoder returns an array of bytes: [...]
[...]
However I am not sure how much the dependency on Foundation.Data is an issue.

If possible avoiding foundation entirely (so also Data) would be good here... Sadly we don't have a different "currency type" for "bag of bytes" today, so that makes interop painful (just array of bytes or NIO types remain). Hopefully we'll address this at some point though; for now you could do the same as Pure does; after all, these types never show up in user API for you right?
However I am not sure how much the dependency on Foundation.Data is an issue.

For example I did not test if, and by how much, it would reduce the binary size, loading time and so.

It is definitely noticeable and painful to even link Foundation, it hurts cold startup time a lot. I don't have the numbers on me right now, but Fabian I think tested that at some point.

@ktoso

If possible avoiding foundation entirely (so also Data) would be good here... Sadly we don't have a different "currency type" for "bag of bytes" today, so that makes interop painful (just array of bytes or NIO types remain). Hopefully we'll address this at some point though; for now you could do the same as Pure does; after all, these types never show up in user API for you right?

if not for the Foundation dependency IkigaJSON seems better choice:

its (a tiny bit but measurably) faster
seems to have better integration with NIO types (BytesBuffer) which I have not checked yet but could potentially bring additional performance boost (?); XRayUDPEmitter does use NIO to send UDP so while dependency on Foundation is disadvantageous, dependency on NIO is not

It is definitely noticeable and painful to even link Foundation, it hurts cold startup time a lot. I don't have the numbers on me right now, but Fabian I think tested that at some point.

@fabianfett
I would be very interested to see any results here.
This is especially important for lambdas.
Maybe will try to play with that a bit in my Swift lambda template/playground.

@fabianfett @ktoso
IMO any swift-server library using JSON encoder/decoder should:

include default implementation
dont say "dont use the default implementation!" (see Swift AWS Lambda Runtime)
considering the issue is not resolved, provide an easy way to change the JSON encoder/decoder

did some reorganization as far aws-xray-sdk-swift is concerned: https://github.com/pokryfka/aws-xray-sdk-swift/pull/42
there is new AWSXRaySDK providing JSON encoder implementation (PureSwiftJSON atm)
planning to do more performance testing of how Foundation affects Lambda cold start
as well as IkigaJSON vs PureSwiftJSON;
IkigaJSON seems to be about 20% faster than PureSwiftJSON atm, not sure how much of an effort is removing dependency on Foundation in IkigaJSON; or perhaps bring in some of the performance from IkigaJSON to PureSwiftJSON

authors: fabianfett ktoso pokryfka

repo: pokryfka/aws-xray-sdk-swift


text:
FIX: honor OMP_NUM_THREADS environment variable

We just hit #844 with mriqc 0.15.2.
Looking at https://github.com/poldracklab/mriqc/blob/master/mriqc/cli/parser.py#L378 we thought we could work around that by setting OMP_NUM_THREADS to 1 while still using values other than 1 for --nprocs. Unfortunately that didn't work because config.nipype.omp_nthreads is always set to os.cpu_count().
This PR checks if the OpenMP environment variable exists and defaults back to os.cpu_count() - which is a confusing choice anyway, because it will always trigger the config warning if someone uses --nprocs other than 1 or os.cpu_count(), right?

Whoops, CI caught a missing int() :pray:

This seems mergeable?

sure, although we would need to check the run-to-run repeatability on circle (see failed run) - if the MD5 sum of some intermediate files are going to be changing, we need to think of a workaround for circle.

The md5sum mismatch is coming up again in #862. It seems likely to be a brittle check...
Anyway, if that is consistent enough to be resolved by patching, I'll try merging this branch into that to test.

Please see #877 for the status of run-to-run repeatability.

authors: effigies octomike oesteban

repo: poldracklab/mriqc


text:
Plane orbit plotter and more

I fixed one last thing that made me unhappy about the latest changes in the orbit plotters: now the plane is a parameter set up in the constructor, not an obscure internal property that is changing along the way
Therefore, I removed the plane parameter from some methods where it didn't make sense anymore
Pluto is not a SolarSystemBody anymore - I mean, it is a Solar System body :wink: but not in the sense we consider it in poliastro

Perhaps we should rename that to SolarSystemPlanet?

Retain linestyle in Trajectoryes so they are used when redrawing
Final refactor in the plotting methods - I'm supper happy :sparkles: of how this ended
Better errors in get_mean_elements
Moved warnings to separate module
Stricter validation of inputs for Orbit creation, fixed #716
Use some more f-strings here and there

Image tests are failing because I brought this from #882 and there is a change of behavior there: ephemerides are plotted as solid lines, but the current implementation of plot_body_orbit creates an intermediate Orbit. There is a trivial way to fix it:
diff --git a/src/poliastro/plotting/_base.py b/src/poliastro/plotting/_base.py
index 30d92e4..a15dea7 100644
--- a/src/poliastro/plotting/_base.py
+++ b/src/poliastro/plotting/_base.py
@@ -142,7 +142,7 @@ class BaseOrbitPlotter:
             coordinates, None, label=str(label), colors=colors, dashed=False
         )

-    def _plot(self, orbit, *, label=None, color=None, trail=False):
+    def _plot(self, orbit, *, label=None, color=None, trail=False, dashed=True):
         colors = self._get_colors(color, trail)

         self.set_attractor(orbit.attractor)
@@ -153,7 +153,7 @@ class BaseOrbitPlotter:
         coordinates = orbit.sample(self._num_points)

         return self.__add_trajectory(
-            coordinates, orbit.r, label=label, colors=colors, dashed=True
+            coordinates, orbit.r, label=label, colors=colors, dashed=dashed
         )

     def _plot_body_orbit(
@@ -168,7 +168,7 @@ class BaseOrbitPlotter:
             warnings.simplefilter("ignore", DeprecationWarning)
             orbit = Orbit.from_body_ephem(body, epoch)

-        return self._plot(orbit, label=label or str(body), color=color, trail=trail)
+        return self._plot(orbit, label=label or str(body), color=color, trail=trail, dashed=False)

     def plot_trajectory(self, coordinates, *, label=None, color=None, trail=False):
         """Plots a precomputed trajectory.

But I won't bother:

implementing this hack, because in #882 it's not necessary anymore
change the images, because I will have to change them again in #882 and this will increase the size of the repository

Python 3.6 and 3.7 passed, merging straight away.

repo: poliastro/poliastro


text:
T. Crear sistema de inserciones en la base de datos

[ ]  Código de inserción para la tabla USUARIOS
[ ]  Código de inserción para la tabla TIPOS_USUARIOS
[ ]  Código de inserción para la tabla EVENTOS
[ ]  Código de inserción para la tabla SERIES
[ ]  Código de inserción para la tabla EVENTOS_USUARIOS
[ ]  Código de inserción para la tabla SEDES
[ ]  Código de inserción para la tabla CIUDADES
[ ]  Código de inserción para la tabla SESIONES
[ ]  Código de inserción para la tabla LOGS
[ ]  Código de inserción para la tabla FACULTADES

Esta tarea ya no aplica, ya que según el proceso que se esté haciendo las consultas pueden cambiar y depende de quien lo esté trabajando, de todas formas  he creado una carpeta Consultas donde se pueden consultar varios ejemplos de consultas usadas en servicios, de igual forma por el canal BD de Slack se atienden dudas acerca de.

authors: degualteros willihamer


text:
Add boolean column for read/not read

It will be useful to add a new column in the Notion database to represent the read/unread status of an article.
When Notion client sends the new page add that new property to the request.

Added new column in Notion database and with default false there's no need to update the call send to Notion via API


text:
Added support for the go to definition functionality

[x] onDidChangeContent is called for every single change, we don't want to find references every time the document changes (but batch them somehow)
[x] change the name of omtLinkProvider (because it does provide more than only links), or split functionality
[x] unittests
[x] move yaml dependency to server
[x] update CHANGELOG.md

Nice.

@emielb I refactored most of the code and split it up into multiple files. I also removed support for declared imports, otherwise I had to make a temporary solution for something that will be removed in the near future.

authors: JordyMackay RogierEnders

repo: politie/omt-odt-language


text:
chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-07-07)
Features

init (d403de8)

Bug Fixes

update template (1e518c1)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:

repo: polyrepos/template-fullstack


text:
community curation totals

The number of community curation was ~12,000 until recently it's now 23,000
Year	Publications curated	Annotations
All	Curator	Community	Curator	Community
Totals: | 3930 | 3061 | 869 | 58350 | 23930
I suspect all of the numbers have somehow doubled?
Or maybe our additions are added to the community curation?

Or maybe our additions are added to the community curation?

I dug into this and that's partly true.  The code for generating these numbers needs improving as it relies on the curator name attached to the annotations to decide which are admin and which are community.  There were about 7700 annotations where the curator name was "Val Wood", "midori" or "antonia".  The code only looks for "Valerie Wood", "Midori Harris" and "Antonia Lock" because the names that Canto has for you in the main database: https://curation.pombase.org/pombe/view/list/person?model=track
I think this might happen if you enter a session and don't put those exact names in when it asks for the curator details.  I plan to re-write this code at some point to make it more sensible.  But for now that's a problem to watch out for.
I've fixed the annotations that have "Val Wood", "midori" or "antonia" as the curator name so the counts are now:

.
All
Curator
Community
Curator
Community

Totals:
3934
3061
873
66092
16482

Great, that makes sense...

authors: ValWood kimrutherford

repo: pombase/website


text:
new search feature at SGD

A sensible implementation of a kw search
http://www.yeastgenome.org/
follow home page link to
http://www.yeastgenome.org/search?q=ethanol&is_quick=true
you quickly realise how sensible your search term is.
Can leverage ontology formal definitions here....
One for discussion....

hmm, my first test gave slightly odd results.
For metabolism (aka metabolic process)
I got 420 genes returned, even though GO  has over 300 SGD annotatins. So it really is a very literal keyword search.
We need to explore this further.
Have a look, and then ignore for now.


text:
Support < (word start) and > (word end) boundaries

Summary
Add two new sigils, < and >, that match the start and end of a word, respectively.
Motivation
In almost all situations, when you use % (a word boundary), you really want to match only a word start or a word end, but not both.
For example: % 'test' % should match the word test, but no other words containing it (e.g. fastest, testament). In this case, it is clear that the first % is a word start and the second % is a word end. However, a more explicit syntax (< and >) would be better for several reasons:

It shows the intent more clearly
It is not susceptible to bugs where % accidentally matches the wrong kind of boundary
It's probably more performant in many regex engines
In JavaScript, where \b is not Unicode aware, polyfilling < and > is much more concise than polyfilling %

Details
Add two new sigils, < and >, that match the start and end of a word, respectively. This means that % is equivalent to < | >. They can not be negated for now.
Whitespace must be taken into account when parsing: << is a lookbehind assertion, but < < should be parsed as two consecutive word starts (which is pretty useless btw).
Pros:

< and > are already established for this purpose in PCRE, GNU and POSIX regex flavors, and now Rust
<'foo'> looks nice and will be easy to remember
<< and >> serve a somewhat similar purpose, so hopefully it will make sense intuitively

Contras:

People might confuse < with <<
It uses the remaining type of brackets. Pomsky already uses [] for character sets, () for groups, and {} for repetitions, so after this there will be no brackets on a typical keyboard left for future syntax additions.

Support in different regex engines
There are two regex flavors supported by Pomsky that natively support word start and word end assertions:

PCRE, using the [[:<:]] / [[:>:]] syntax. The syntax is borrowed from POSIX regular expressions.
Rust, using the \< / \> syntax (supported since version 0.10). The syntax is borrowed from GNU regular expressions.

Word start and word end assertions can be polyfilled in regex engines that don't support them natively:

< becomes (!<< [word]) (>> [word])
> becomes (<< [word]) (!>> [word])

We also plan to implement optimizations that simplify the expression, and one possibility is to remove redundant assertions. For example, (>> [word]) can be removed if we know that the assertion is followed by a word character. So < 'test' could be compiled to just (?<!\w)test instead of (?<!\w)(?=\w)test.
Alternatives
% could be optimized into (!<< [word]) or (!>> [word]) when possible, by looking at the surrounding characters. However, this addresses only half of the benefits (3. and 4.) addressed by proposal.
The assertions could be implemented as keywords or as built-in variables, e.g. WordStart 'test' WordEnd. However, this is much more verbose than the proposed < and > symbols.
In the same fashion as Rust, we could allow %{start} and %{end}. Again, this is much more verbose than < and >. The nice thing about this syntax is that it emphasizes the similarity to %. It also allows us to support %{start-half} and %{end-half} (not covered by this proposal).

Except for optimizations, this is done in f5266fa

repo: pomsky-lang/pomsky


text:
Refactoring: boolean evaluation

https://github.com/poneoneo/Alibaba-CLI-Scrapper/blob/9c7c45c6cd15c79fa965931bdbc2ecd176593ada/aba_cli_scrapper/app.py#L35
if <something> is True: <=> if something:

i'm planning to remove this file it will no longer be available. main code is now in main.py file. and for yur suggestion some time im telling me that is more readable.

authors: poneoneo yoyonel

repo: poneoneo/Alibaba-CLI-Scrapper


text:
Disable avx512f for x86 targets only

Hi, this PR is trying to fix minor compilation issues I've found building and using ponyc on arm (32bit only at the moment), I hope I haven't done anything completely wrong.
Currently -avx512f is added explicitly to the list of cpu features in libponyc/codegen/host.cc and libponyc/codegen/codegen.c for all platforms when llvm < 5.0.0, though it is only supported by x86. As a result ponyc is generating at least one warning '-avx512f' is not a recognized feature for this target (ignoring feature) at multiple stages.
Plus there's a minor bug in line 789 of src/libponyc/codegen/codegen.c
size_t temp_len = strlen(opt->features) + 9; // should be 10 instead
char* temp_str = (char*)ponyint_pool_alloc_size(temp_len);
snprintf(temp_str, temp_len, "%s,-avx512f", opt->features);

And running ponyc with a set of features (on all platforms) like ponyc --features=+sse2 will give '-avx512' is not a recognized feature for this target (ignoring feature) as f in -avx512f was truncated.
Last issue is -mcpu=$(arch) won't work for arms most of the time if arch is redefined, e.g. for my armv7 target I need to set march=armv7, mtune=cortex-a9 and mcpu=cortex-a9. Setting cpu to armv7 (=arch) isn't correct and llc will log lots of warnings. I believe the simplest solution is to make cpu variable configurable.

Thanks @ikavalio! Once this is merged, its going to be a really nice user experience improvement.

Thanks a million for the review! I've tried to fix code style issues and replaced custom target check with target_is_x86 function.

authors: SeanTAllen ikavalio

repo: ponylang/ponyc


text:
Update String documentation to be more correct

String is really a collection of bytes and several methods assume
that characters are a single-byte.

I think this is probably less confusing. Thoughts @jemc @Theodus ?

@jemc any thoughts before this is merged?


text:
[Feature] If user is logged in with google we can display profile picture from it

Description
@pooranjoyb
if user logged in with google and it have profile picture set in google account
we can display it in profile image
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Congratulations, @dhruv8433! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Yup go ahead integrate that too :) @dhruv8433

repo: pooranjoyb/popShop


text:
fix: detectOverflow export

@lainermeister does this fix the issue?
Closes #1130

@lainermeister does this fix the issue?

I'm not able to test as it doesn't seem like 2.4.3 is released yet!

authors: atomiks lainermeister

repo: popperjs/popper-core


text:
Per sample size endpoint

Endpoint to return the per project, per sample size for each sequence type over a set date range.

Codecov Report

Merging #273 (01d8440) into dev (465be12) will decrease coverage by 0.64%.
The diff coverage is 31.00%.

@@            Coverage Diff             @@
##              dev     #273      +/-   ##
==========================================
- Coverage   64.03%   63.38%   -0.65%
==========================================
  Files          65       65
  Lines        4957     5045      +88
==========================================
+ Hits         3174     3198      +24
- Misses       1783     1847      +64

Impacted Files
Coverage Δ

db/python/tables/analysis.py
22.75% <0.00%> (-0.42%)
:arrow_down:

db/python/layers/analysis.py
31.91% <15.38%> (-12.73%)
:arrow_down:

api/routes/analysis.py
41.21% <21.42%> (-5.94%)
:arrow_down:

db/python/tables/sample.py
27.86% <25.00%> (-0.07%)
:arrow_down:

api/routes/sample.py
43.61% <33.33%> (-0.71%)
:arrow_down:

db/python/layers/sample.py
50.00% <50.00%> (ø)

db/python/layers/web.py
40.29% <50.00%> (+0.59%)
:arrow_up:

models/models/analysis.py
75.67% <100.00%> (+11.67%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

authors: codecov-commenter violetbrina

repo: populationgenomics/sample-metadata


text:
support layout variants

This PR makes it possible to switch between layouts with different layout variants.
Example usage:
path/to/i3-keyboard-layout cycle "bg bds" "bg phonetic"
or
path/to/i3-keyboard-layout cycle us "bg phonetic"

Great stuff, it works perfect! Thak you :smile_cat:

authors: porras slaviber


text:
ramp~

[del 0] into the object
bang in clip mode

get both sweeper~ and phasor~ from superclonider and make it into a ramp~ object
it's kinda like counter~ but triggered with triggers/impulses (counter is "latch")
-1 trigger stops it
square could also drive it... like train does drive count

ramp~ is too complicated already... if you change it, make it simpler!!!

Add bang outlet for ramp~ and susloop~ (like Line~)
only not, add impulse output when reaching limits

create line2~ or ramp~ for time input in ms
create a variaton of cyclone/wave~ (sampler~?) with index input
wavetable~... (tabosc4~) with susloop

repo: porres/pd-else

license: WTFPL


text:
[dir]: cannot open

When I create a object dir.
[dir]: cannot open 'C:/Users/xxx/jlquispec', opening 'C:/Users/xxx/jlquispec' instead
elsev1.0beta27.dek
Uploaded by porres @ 2020-02-06 17:36:29
Windows 10
Pd version 0.50.2

this has already been fixed, wait for next release (beta 28)
thanks for reporting

authors: jlquispec porres


text:
Can I use this manager to transfer files internally to volumes

Question:
How can I deploy Portainer on... ?
Hello,This manager is very easy to use, I would like to ask if there is such a function
Can I use this manager to transfer files internally to volumes

If you are running the Portainer agent and have added it as an endpoint, then volumes should show the browse button in the UI. When browsing a volume like this, you can then upload files

Yes, but only if you deploy Portainer in conjunction with our agent, and then enable host management features in settings.

Thank u very much..I will try

Linknto documentation on the agent

I'll close this since we have answered the question. Any further problems we can discuss on this issue, or alternatively you can reach us on slack here

Thanks again for your help, I encountered a new problem, when the uploaded file is too large, an exception will occur：
Invalid file path

authors: itsconquest ncresswell vast-z

repo: portainer/portainer

license: Zlib


text:
Restart pds application pod during resource update

This PR has the test RestartAppDuringResourceUpdate
Deploy data service
Update the resource config, while upgrade is happening delete few pods.
Validate post deletion all the pods come up successfully.

Add test execution logs in description

repo: portworx/torpedo


text:
Add Visual studio solution structure

Possibility to open in Visual Studio ( still able to open in Visual Studio Code)

Thanks!

repo: porumacausa/bom-vizinho-api


text:
Add support for Shiny Express in Quarto Dashboards, take 2

This PR supersedes #1212. It adds support for Shiny Express apps in Quarto Dashboards. Prior to this PR, it would throw an error if a Quarto document had an import from shiny.express.
For example:
---
title: "Palmer Penguins"
format: dashboard
server: shiny
---

```{python}
#| context: setup
import seaborn as sns
from shiny.express import render, ui, input
penguins = sns.load_dataset("penguins")
```

# {.sidebar}

```{python}
species = list(penguins["species"].value_counts().index)
ui.input_checkbox_group(
    "species", "Species:",
    species, selected = species
)
```

# Plots

```{python}
@render.plot
def depth():
    data = penguins[penguins["species"].isin(input.species())]
    return sns.displot(
        data, x = "bill_depth_mm",
        hue = "species", kind = "kde",
        fill = True
    )
```

This is what it looks like:

The generated app.py looks like this.
# This file generated by Quarto; do not edit by hand.

from __future__ import annotations

from pathlib import Path
from shiny import App, Inputs, Outputs, Session, ui
import shiny.express

shiny.express.allow_express_in_core = True

import seaborn as sns
from shiny import reactive
from shiny.express import render, ui
penguins = sns.load_dataset("penguins")

# ========================================================================

def server(input: Inputs, output: Outputs, session: Session) -> None:
    # from shiny.express import render, ui
    species = list(penguins["species"].value_counts().index)
    ui.input_checkbox_group(
        "species", "Species:",
        species, selected = species
    )

    # ========================================================================

    @render.plot
    def depth():
        data = penguins[penguins["species"].isin(input.species())]
        return sns.displot(
            data, x = "bill_depth_mm",
            hue = "species", kind = "kde",
            fill = True
        )

    # ========================================================================

    return None

_static_assets = ["simple_files"]
_static_assets = {"/" + sa: Path(__file__).parent / sa for sa in _static_assets}

app = App(
    Path(__file__).parent / "simple.html",
    server,
    static_assets=_static_assets,
)

A few things to note about this:

It uses the UI was generated by Quarto (in the form of a static .html file).
This change allows a shiny.express import to be used in a Shiny Core app, if the code contains shiny.express.allow_express_in_core = True at the top level. Note that this statement is used in two different ways:

Once when is_express_app() scans the AST of the app code. It looks for that specific statement at the top level.
Once when actually running the app -- it tells run_express() to not throw an error when it encounters the shiny.express import in a Shiny Core app.

The is_express_app() function has changed. There are other places where we use copies of this code, (Shiny Server and rsconnect-python), and the function will need to be updated in those places.

Use a magic comment to force app to be treated as Core or Express app.
Warn if page_opts() or app_opts() are called?

repo: posit-dev/py-shiny


text:
⚠ ku-s has degraded performance

In a52a288, ku-s ($KU_API_STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 18779 ms

Resolved: ku-s performance has improved in 6ab647a after 49 minutes.

repo: positive-infinity/api-status


text:
Coloring the letters in sections in different colors?

As the title suggests: I would like to give the 3 letters of the sections 3 different colors. For example: the first letter in red, the second in blue and the last in green.
Terrific job btw.

I don't understand why you're considering this, but if you really want to this below is the simplest way I could conceive of doing it.
It requires a few minor edits to  awesome-cv.cls.
I've included some edits for resume.tex (and cv.tex and coverletter.tex if you intend to use them as well) to make the modification easier to work with.
Modifying Section Color Behavior in awesome-cv.cls
Within awesome-cv.cls change the \ifbool under % Awesome section color from this:
\ifbool{acvSectionColorHighlight}{{\color{awesome}#1#2#3}}{#1#2#3}
to
\ifbool{acvSectionColorHighlight}{{\color{awesomeSecOne}#1}{\color{awesomeSecTwo}#2}{\color{awesomeSecThree}#3}}{#1#2#3}
This change allows for each of the colors of the first three letters to be defined individually.
Defining Colors in Appropriate .tex Files So You Can Leave the .cls file Alone
Within the appropriate .tex file (resume.tex, cv.tex, and/or coverletter.tex), presumably with the other color definitions, add the following:

\definecolor{awesomeSecOne}{HTML}{789455}
\definecolor{awesomeSecTwo}{HTML}{145876}
\definecolor{awesomeSecThree}{HTML}{445566}

Adjust the HTML color codes to whatever you want. If you want to use the predefined awesome colors you can find their html color codes within the .cls file.
Note: For the sake of completeness...\definecolor{awesomeSecThree}{awesome-red} will not work but \colorlet{awesomeSecThree}{awesome-red} will.

Thank you @dcsim! I don't think this is something that will be natively supported, at least not without a much broader scope overhaul, so I'll close this now grateful for your explanation on OP's specific request.

authors: OJFord dcsim me7pako


text:
Adding Teachers (BULK ENTRY)

A bulk entry option is needed

Teacher self sign up will be used.


text:
docs(library): document pgSettings option

I spent quite some time looking for this. I thought it'd be useful to have it in the documentation.

This is good to merge; I can't debug CI from the train, alas.

Thanks @florianherrengt 🙏

authors: benjie florianherrengt

repo: postgraphql/postgraphql


text:
Encoding

Thanks so much, @vjyanand. Is there any chance you could add a test verifying the fix so that in the future we can more easily catch a regression? There are a handful of encoding-related tests in src/resource/index.test.js, so that would be a great place to test this one.
Thanks again!

Closing in favor of #270. Thanks for getting the ball rolling here!

authors: adampash vjyanand

repo: postlight/mercury-parser


text:
Basic backend

Have a more basic backend without multishot that only adds atoms and grounds every time

Closed with [#131]

repo: potassco/clinguin


text:
Predicate docstring in same block

If the docstring of two predicates is not in the same comment block then only the first on is obtained

If it is -not - in the same comment block? Or rather if it - is - in the same comment block?
The current implementation assumes one comment-block per docstring.
Case 1: this should work
%*
#some_predicate(A,B,C).
some description|
*%
%*
#some_other_predicate(A,B,C).
some other description|
*%

Case 2: this should not work and likely results in an odd predicate documentation
%*
#some_predicate(A,B,C).
some description|

#some_other_predicate(A,B,C).
some other description|
*%

If we want to support case 2 as well (which seems somewhat unusual right? Usually there is one doc string per struct/function/field/whatever), then I think we would have to define some kind of delimiter in order to cleanly separate the different predicates.
Because we have to then differentiate between the start of a new predicate description and the continuation of the previous predicate.
If you're not talking about case 2, can you give an example?

authors: VictosVertex susuhahnml

repo: potassco/mkdoclingo


text:
Fixes #102, don't prepend all keys with "event"

I'm included the documentation in this PR because it's critical that users know NOT to use the documented GA keys eventCategory, eventAction, eventLabel, and eventValue, because the ga adapter does that for you.
That little example would've saved me so many hours of digging around..

:umbrella: The latest upstream changes (presumably 663c7ad) made this pull request unmergeable. Please resolve the merge conflicts.

repo: poteto/ember-metrics


text:
Use with CubicSDR problems

looking forward to get this updated version running but it seems that it will only build against the as yet unreleased v0.6 SoapySDR as it uses the new KwargsToString API which was added after the 0.5.4 release. CubicSDR 0.2.0 is also built against SoapySDR 0.5.2 so I presume the only way of getting CubicSDR to use this is to build it from source?
Thanks
Richard

@gedger yup -- at the moment I'm building CubicSDR against v0.6; and will release some bookmarking test builds with updated soapy mods for OSX, Windows and Linux shortly.

It should be ok now to compile against releases of soapysdr: https://github.com/pothosware/SoapySDRPlay/commit/d7eb5d7b0e462160b4fb2977da3f52d5c228dfaa

authors: cjcliffe gedger guruofquality

repo: pothosware/SoapySDRPlay


text:
Not working with relational-pouch

I can see that it works with default db.put() as in the docs, but then when i try using relational-pouch to create documents, i get no results for the same data.
See example here: https://runkit.com/rikki/pouchdb-experiments

Just came across this same situation, and: relational-pouch puts all document data in a "data" key, so instead of fields: ['stuff'] it should be fields: ['data.stuff'].

authors: BrunoDeBarros acao

repo: pouchdb-community/pouchdb-quick-search


text:
Implement Multi-return

Allows an instruction to return more than one field element, see example in the new multi_return test.

Is this A, B <=X= 1 used anywhere?

Looks good! Please rebase.

two small items but I think they should be changed, good to merge after imo

authors: Schaeff chriseth georgwiese leonardoalt

repo: powdr-labs/powdr


text:
result_inline.html character encoding issue

Copied from wq/wq.db#76:

When creating a campaign with a field name containing a single quote(ex: Child's Name), the rendered label in result_inline.html partial doesn't encode the quote properly(ex: Child's Name). However the description containing the same text renders correctly(ex: Child's Name).

It might be possible to fix this by updating the template to use triple Mustache bracket which will trigger verbatim rendering ({{{type_label}}}).
Note that I am in the process of updating several dependencies and I will revisit this app once that is done. (See wq/vera#5)

The triple mustache bracket does indeed fix the HTML rendering.  In the future, I feel like we might want to address the issue server-side.  /reports/new.json returns the html-escaped label format(Child's Name) and probably shouldn't.

Ok, I think once wq/vera#6 is merged, this can be closed.

authors: sheppard tubaman


text:
context menu issue

started happening recently after a discord update which broke other plugins too

I'll work on it when I'm free.
probably the next few days.

@xBiei thank you very much, i sold the push which probably fixes it

For the future of everyone else, I'm tired of supporting this shit, maybe this is the last update

this also started to happen recently on right click of any user's profile:
Error: Failed to patch image-tools-guild-channel-user-context-menu; module is invalid.
    at inject (kernel://K:\Downloads\packages\pc-compat/dist/renderer.js:1:61274)
    at inject2 (K:\\Downloads\\powercord\\plugins\\image-tools\\utils\\inject2.js:26:3)
    at General.injectWithSettings (K:\\Downloads\\powercord\\plugins\\image-tools\\modules\\Patcher\\General.js:368:16)
    at eval (K:\\Downloads\\powercord\\plugins\\image-tools\\modules\\Patcher\\General.js:87:22)
    at https://discord.com/assets/0eb297585b87bfddc0a2.js:757:158
    at t.n.render (https://discord.com/assets/0eb297585b87bfddc0a2.js:3553:703)
    at qa (https://discord.com/assets/0eb297585b87bfddc0a2.js:5886:489)
    at Xa (https://discord.com/assets/0eb297585b87bfddc0a2.js:5886:288)
    at $s (https://discord.com/assets/0eb297585b87bfddc0a2.js:5926:166)
    at ku (https://discord.com/assets/0eb297585b87bfddc0a2.js:5916:748)```

this also started to happen recently on right click of any user's profile:

Error: Failed to patch image-tools-guild-channel-user-context-menu; module is invalid.

    at inject (kernel://K:\Downloads\packages\pc-compat/dist/renderer.js:1:61274)

    at inject2 (K:\\Downloads\\powercord\\plugins\\image-tools\\utils\\inject2.js:26:3)

    at General.injectWithSettings (K:\\Downloads\\powercord\\plugins\\image-tools\\modules\\Patcher\\General.js:368:16)

    at eval (K:\\Downloads\\powercord\\plugins\\image-tools\\modules\\Patcher\\General.js:87:22)

    at https://discord.com/assets/0eb297585b87bfddc0a2.js:757:158

    at t.n.render (https://discord.com/assets/0eb297585b87bfddc0a2.js:3553:703)

    at qa (https://discord.com/assets/0eb297585b87bfddc0a2.js:5886:489)

    at Xa (https://discord.com/assets/0eb297585b87bfddc0a2.js:5886:288)

    at $s (https://discord.com/assets/0eb297585b87bfddc0a2.js:5926:166)

    at ku (https://discord.com/assets/0eb297585b87bfddc0a2.js:5916:748)```

Yeah that's due to the new discord changes to some context menus

the issue still hasn't been fixed it seems

oh it's still happening to me after updating is there any way that i can resolve this issue this is what i see on my dev console

looks like the issue persists even after the fix. I'll check what's the problem.

oh thanks!

I guess it's time to switch to the new context menu injection instead of this one.
it needs some refactoring so it'll take some time cuz I'm not entirely free.
unless xinos is gonna work on it.
if you urgently need this you can use lighty's using bdcompat.

authors: Mertcan213 kekkodance userXinos xBiei

repo: powerfart-plugins/image-tools


text:
Update golang/master build

This is an automated PR via build-bot

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ltccci
To complete the pull request process, please assign  after the PR has been reviewed.
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
是否有人曾提过类似的问题？
否（No）
你觉得APP有什么不足之处？
我个人开启了转发短信和只转发88888888包名的电量，我对于 https://github.com/pppscn/SmsForwarder/commit/c2df047dc1b36f7ed24559adca823b8c7a84c53e 的理解是com.android.mms包名并没有被匹配，造成短信通知不会被自动清除了
你觉得该怎么去完善会比较好？【非必答】
临时解决是关闭短信转发并开启com.android.mms和88888888包名转发
似乎可以把com.android.mms包名直接内置在自动清除的内部？

短信广播的形式转发短信，却想要消除 短信通知 是吗？

是的，外加不同ui短信包名不同和有人用第三方短信，内置包名估计行不通

下载每周构建版测试一下
https://github.com/pppscn/SmsForwarder/actions/workflows/Weekly_Build.yml
【注意】因为去除了 mmkv 依赖，采用 SharedPreferences 来保存配置，建议测试前离线导出配置，然后全新安装后重新导入配置

repo: pppscn/SmsForwarder


text:
Re-calling of browse() by ref does not work

I needed to move the browse button beyond the Filepond component. In a big simplification, I did it so that in vuex I change the status of eg showBrowse after clicking the button.
The script from Filepond observes (watch) the state showBrowse = TRUE, then runs this.$refs['pond'].browse()
It works. The problem occurs when the <file-pond> is being wrapped in the display object depending on the value, eg showFilepond
<div v-if="showFilepond">
  <file-pond ref="pond" ... />
</div>

showFilepond == FALSE - after sending files
After the first launch the upload works correctly. When you change the showFilepond to TRUE again, the browser opens, but the selected file is not sent (does not appear in the list)
To make it more interesting, after clicking the standard link in the component (open the browser), after selecting the file, everything works fine.
However, selecting the file by the browser running by $refs does not work, why and how to fix it?

From what I've found yet. If the same filepond object (the same ref identifier) is displayed again, the problems are as above.
As before initializing the object I will generate a new ref key and give it to filepond, the problem will be solved.
Why it does not work on the same ID ?

I've just read the issue and am sorry but I don't understand the exact issue.

If I find a free moment, I will prepare an example for you.

Closed in the mean time

authors: alorenc rikschennink

repo: pqina/filepond


text:
feat: add gauge icon

https://github.com/user-attachments/assets/000226ae-2957-4c3d-b3ba-062eb963eb22

i’ll go ahead and merge this PR and handle the last few fixes. need it in place to release the search update since the structure’s been reworked

repo: pqoqubbw/icons


text:
Windows support?

Now that Meteor is about to release official support for Windows, would there be interest in making spacejam run on Windows? I know it's intended for CI, which runs on *nix machines, but for local development, it would be useful to have Windows support. Right now, I get
> spacejam --mongo-url mongodb:// test-packages ./

spacejam: spawning meteor
spacejam: killing meteor

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: spawn ENOENT
    at errnoException (child_process.js:988:11)
    at Process.ChildProcess._handle.onexit (child_process.js:779:34)

Would be nice, but I won't work on this one until windows support is officially out there and stable.

Got the same issue? Any plans on supporting Windows?
The above issue is easily fixable just by using cross-spawn here https://github.com/practicalmeteor/spacejam/blob/master/lib/ChildProcess.js#L14 instead of child_process.
However I did not have time to check if any other problems will not pop up after that one.

@wojtkowiak thanks for the pointer, will check it out.

Hi,
It's working great on windows with this commit, but I didn't run tests so tell me if it's still ok.
As you know, psargs and ppid are not supported under windows, so I needed to remove this test https://github.com/serut/spacejam/commit/ffaa9c9d26d8ed60d83f6315350743082786799b#diff-0b5f1f297c7739069dfef6aa7cc959c1L29. Is it really important to check that mongod has the PPID of meteor ? Another solution would be to edit practicalmeteor/ps to ignore psargs and ppid if the  running platform is windows.
Renaming Phantom.js into PhantomjsRunner.js was mandatory, I don't know why cross-spawn.spawn first looks inside the directory if a file match and then in the PATH, but it's what I observed.

I had this issue as well (Windows 10) and wanted to fix it because my Cloud9 *nix VM tends to run out of disk space so I need an alternative local IDE and meteor installation.
Running meteor test-packages runs fine out of the box btw.
So here's how I got it to work (it's not exactly the same as the commit does):

made backups of all the files in my npm\node_modules\spacejam\lib folder
use cross-spawn-async in ChildProcess Line 14

ChildProcess._spawn = require("cross-spawn-async").spawn;

added Windows-specific code to findAllChildren function in MeteorMongodb.js Line 40:

if (process.platform === 'win32') {
  var resultList = [];
  var bat = require('child_process').spawn('cmd.exe', ['/c', 'getChildren.bat ' + this.meteorPid]);

  bat.stdout.on('data', function (data) {
    var childPid = data.toString().trim();
    resultList.push({pid: parseInt(childPid)});
  });

  bat.stderr.on('data', function (data) {
    return log.warn("spacejam: Warning: Error enumerating mongod children:\n", data);
  });

  bat.on('exit', function (code) {
    if (code != 0) {
      return log.warn("spacejam: Warning: Enumerating mongod children returned with error code: ", code);
    }
    log.info("MongoDB children:\n", resultList);
    this.mongodChilds = resultList;
    if (resultList.length === 0) {
      return log.warn("spacejam: Warning: Couldn't find any mongod children:\n", err);
    } else if (resultList.length > 1) {
      return log.warn("spacejam: Warning: Found more than one mongod child:\n", resultList);
    } else {
      return log.debug("Found meteor mongod child with pid: ", resultList[0].pid);
    }
  });

  return;
}

created a batch file getChildren.bat to implement getting child processes because *nix ps doesn't cut it on Windows:

@echo off & setlocal enabledelayedexpansion
set PPID=%1
for /F "skip=1 tokens=1" %%a in ('wmic process where "ParentProcessID=%PPID%" get processid') do for %%b in (%%a) do (
  echo %%a
)

I did NOT have to rename Phantomjs.js to PhantomjsRunner.js
I tested with and without renaming and could not see a difference.
what I did have to do was install phantomjs globally (npm i -g phantomjs) because otherwise Spacejam failed to spawn it. It seems just installing spacejam with -g does not suffice because phantomjs does not end up being on the PATH, resulting in the dreaded "spawn ENOENT" error.

After those steps, spacejam ran my package test with test-in-console driver as normal. I ran spacejam with the "--loglevel debug" option to see what's happening. I'm still not sure everything works correctly and I still get "spacejam: Unknown error with exit code 'null'. Exiting." after the tests have run.
Also the spawned node and mongod processes still hang around until I double Ctrl-c in the console, so the code I added might not be enough although it duplicates the ps.lookup behaviour. If anyone finds out more about it, please share your findings. For now I'm happy the tests work. All of this was on Meteor 1.2.1 platform and I'm switching to Meteor 1.3 and refactoring my packages to use module imports/exports. After that, spacejam might not be needed anymore, but we'll see. In any case this post  might be useful for other scenarios that need Windows support...
As an aside, you can slightly modify the batch file to get the parent of a process instead, like this:
@echo off & setlocal enabledelayedexpansion
set PID=%1
set PPID=undefined
for /F "tokens=2 delims==" %%a in ('wmic process where "ProcessID=%PID%" get parentprocessid /value') do (
  if "%PPID%"=="undefined" set PPID=%%a
  goto break
)

:break
echo %PPID%

I put those batch files in the root of my meteor project, but they could be put somewhere else as well I guess, as the spawn function also has options to specify path and working director and whatnot.

I did NOT have to rename Phantomjs.js to PhantomjsRunner.js I tested with and without renaming and could not see a difference.
what I did have to do was install phantomjs globally (npm i -g phantomjs) because otherwise Spacejam failed to spawn it. It seems just installing spacejam with -g does not suffice because phantomjs does not end up being on the PATH, resulting in the dreaded "spawn ENOENT" error.
Yes windows looks first in the directory and then in the PATH ...! But I found an alternative, I use the full path of phantomjs:
https://github.com/practicalmeteor/spacejam/compare/master...serut:master#diff-46f7a2c55394d7160b95c6cfd50c64b9R40
Note that my repository works great on windows, but it doesn't look anymore if mongod is a child of meteor

Thanks @schoepke  and @serut I'm working with windows support and your contributions are welcome, if you want you can create a pull request.
@schoepke you can use spacejam for your app tests as well, the latest release of spacejam has support meteor test command.

Hello @serut @schoepke @wojtkowiak  @dandv  I've just release a new version of spacejam that can support windows, I've include all your suggestions.
To try the new version run this:
npm install -g spacejam@1.6.2-rc.1

It's not working for me.
I'm getting the following error:
Error: Cannot find module 'coffee-script/register'     at Function.Module._resolveFilename (module.js:325:15)     at Function.Module._load (module.js:276:25)     at Module.require (module.js:353:17)     at require (internal/module.js:12:17)     at Object.<anonymous> (C:\Users\bruno\AppData\Roaming\npm\node_modules\spacejam\bin\spacejam:3:1)     at Module._compile (module.js:409:26)     at Object.Module._extensions..js (module.js:416:10)     at Module.load (module.js:343:32)     at Function.Module._load (module.js:300:12)     at Function.Module.runMain (module.js:441:10) Exited with code: 1. Error executing child process: Error: Command failed: C:\WINDOWS\system32\cmd.exe /s /c "spacejam -mongo-url mongodb:// test-packages ./meteor" module.js:327     throw err;

authors: brunoalvesdefaria dandv jsep rbabayoff schoepke serut wojtkowiak

repo: practicalmeteor/spacejam


text:
Null Reference Exception by button click

Dear Ooui Team,
ive bind over XAML a Button via a Command, as it  do not work, so i bind over x:name the button to a click method. Same error its happen. The error in my Debugger console on Chrome is:
Failed to process received message: Newtonsoft.Json.JsonSerializationException: Error getting value from 'MessageType' on 'Ooui.Message'. ---> System.NullReferenceException: Object reference not set to an instance of an object.
thrown in mono.js
i use VS Studio 2017 and the newest (released) nuget packages for .net Standard and Xamarin.Forms.
The over bindings and Code behind Systems work for me. Just the Command pattern seems to be buggy.

Did you manage to get this working? i'm getting the same error, simply trying to run basic ooui.forms example with button click.

authors: MartinPessoa rktherk

repo: praeclarum/Ooui


text:
Upgrade molo.core to 6.6.2

https://github.com/praekelt/molo/compare/6.5.0...6.6.2

@alexmuller please use 6.6.2 I have fix the migration issue in 6.6.0 in 6.6.2

@saeedmb8 Sure thing, done!

@alexmuller I made a naming mistake in my migration :( I renamed it and created a new release. now the correct one is 6.3.3. I will Update your PR as soon as the travis build is passed

And I closed your PR 😭

That's ok, @codiebeulaine has bumped to a later version in her recent PR.

repo: praekelt/molo-gem


text:
Only deploy to pypi once per tagged build

Currently Travis tries to deploy to PyPI for both jobs that it runs each build(one for each python version). This is causing the slower job to fail because PyPI complains that the version already exists (because the faster job already uploaded it).

ping @JayH5 are you happy for me to merge this?


text:
different email for last low credit notification

The email that is sent for the lowest notification should be different than the rest, as the last notification is the credit cutoff, and no more messages can be sent after this. The email should reflect this.

Ready for review.

One minor comment, otherwise looks good to me.

Changes made. Ready for re-review.

:+1: from me. I guess we should wait to see if @SteveBarnett has any comments though.

@SteveBarnett I've implemented your suggestions. Have a look and see if you're happy with it.

@rudigiesler Fixed my typos (sorry!). :+1:

:+1: once Travis is happy.

authors: SteveBarnett hodgestar rudigiesler

repo: praekelt/vumi-go


text:
Use new worker logger

Use the new worker logger for a few transports.

Ready for review.

Some of the updated multiline log calls are probably pep8 violations.

Error from Travis:
Traceback (most recent call last):
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/twisted/internet/defer.py", line 1128, in _inlineCallbacks
    result = g.send(result)
  File "/home/travis/build/praekelt/vumi/vumi/transports/smpp/tests/test_protocol.py", line 332, in test_bind_transmitter
    protocol = yield self.get_protocol(bind_type='TX')
  File "/home/travis/build/praekelt/vumi/vumi/transports/smpp/tests/test_protocol.py", line 76, in get_protocol
    dummy_service = DummySmppService(self.clock, self.redis, cfg)
  File "/home/travis/build/praekelt/vumi/vumi/transports/smpp/tests/test_protocol.py", line 28, in __init__
    self, config.delivery_report_processor_config)
  File "/home/travis/build/praekelt/vumi/vumi/transports/smpp/processors/default.py", line 85, in __init__
    self.log = transport.log
exceptions.AttributeError: 'DummySmppService' object has no attribute 'log'

Fixed the pep8 violations. Tests should also be fixed now.

Ready for review again.

:+1: once Travis is happy. I restarted the failed test (it looks unrelated, but wasn't 100% sure).

authors: hodgestar jerith rudigiesler

repo: praekelt/vumi


text:
HealthCheck endpoint configurability

Currently, the healthcheck endpoint checks both rabbitmq and redis, and this will be fetched from CELERY_BROKER_URL.
But in practice, only one of these two will be used.
Currently, this means that one of the checks will always be failing. This is an issue, because the endpoint returns a 500 error if any of the checks are failing, so we can't use any automated checking tools that will just check the status code.
I think there are 2 ways to go about this:

Have separate REDIS_URL and RABBITMQ_URL configs, and choose the BROKER_URL from one of them (preferring one over the other if both are supplied), and only add a healthcheck for each of those if they're supplied.
Have separate BROKER_URL, REDIS_URL, and RABBITMQ_URL, and use the celery healthcheck to check the BROKER_URL, and the redis and rabbitmq healthchecks to check the others, also only adding the healthchecks if the config values are supplied.

I think the issue for 2 is that if the celery worker is down, we don't want to be alerting or restarting the django web process, because there's nothing wrong with it, the issue is with the celery worker. But we still want to be able to alert if Django cannot contact the message broker. For that reason I'm leaning towards 1.

I think adding either ofthe test options to INSTALLED_APPSin dev and production settings with respectable URLs will be the cleanest solution

authors: cnsr rudigiesler

repo: praekeltfoundation/healthcheck


text:
Draggability Options

An option to disable draggability would be useful for different implementations.

@dnusca If you provide an empty dragHandler, it should work fine.
handleDrag: function() {
  // do nothing
}

render: function() {
  <ReactTags ... handleDrag={this.handleDrag} />
}

Do you have any specific ideas about disabling drag?

@prakhar1989 Sorry, I thought the move cursor was implemented in the component. That works perfect.

No problem! :+1:


text:
Added hello world in bash

Added hello world in bash using echo command.
Signed-off-by: Eduardo Henrique edu.hen.fm@gmail.com

Thank you

Add your name in the contributing,md file, add a readme.md to the new directory and we are good to go

Added in #13

authors: ed-henrique pranav-nb

license: Unlicense


text:
Discussion: Could we split information for each kata to enable both experienced and inexperienced users to get the best experience

Each kata could contain:

Problem description
Small hints
Bigger hints
Walkthrough like today ( the exercise part )
Solution

This might give both reusabilty and scalability in the classroom?
Thoughts @meekrosoft @sofusalbertsen ?

I am absolutely in favour of more and "layered" instructions. Some of the current ones are very much on the "thin" side.

I'm all in favour.

Great! Then we just need to do it 😃

authors: JKrag RandomSort meekrosoft

repo: praqma-training/git-katas


text:
Use DevCamper logo in header

Use DevCamper logo in header

logo used and added in header.jsx

repo: prasadhonrao/devcamper


text:
Update 404-error.html

I have updated the link

this is not what I required 🙄😐


text:
🛑 ChatGPT Beyond 2021 is down

In 301d4a2, ChatGPT Beyond 2021 (https://chatgpt-beyond-2021.onrender.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: ChatGPT Beyond 2021 is back up in 12dedb4 after 7 minutes.

repo: prateekralhan/status


text:
🛑 ChatGPT + DALL-E is down

In c6d3eb8, ChatGPT + DALL-E (https://streamlit-chatgpt-dalle2.onrender.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: ChatGPT + DALL-E is back up in 8d37072 after 6 minutes.


text:
🛑 Bangalore - Realtime Weather Stats is down

In b70c7f4, Bangalore - Realtime Weather Stats (https://bangalore-weather-stats.onrender.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Bangalore - Realtime Weather Stats is back up in d928f07 after 7 minutes.


text:
🛑 ChatGPT Beyond 2021 is down

In 9a500f7, ChatGPT Beyond 2021 (https://chatgpt-beyond-2021.onrender.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: ChatGPT Beyond 2021 is back up in c259e38 after 1 hour, 9 minutes.


text:
U

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

create another issue providing more information

repo: prbhtkumr/PhoneSploit


text:
Simplify + clean up class creation

Another option here would be to just use ES Classes directly. Babel transpiles them to a syntax that doesn't work with Custom Elements, which limits options.

CI errors seem to be a browserstack API key issue.

superceded by #21

repo: preactjs/preact-custom-element