Multiple package.json cannot be updated

Hi there,
I have various projects with multiple package.json files (server API / client front-end).
Is there a way to activate version-inc ?
Thanks,
Costis

So sorry that I didn't see this sooner. I have been without a computer for quite a while but that has finally changed. You may have figured this out already but you can add as many files to be updated as you want. Use the command Version Inc: Edit Settings for Update Files from the command palette and edit the file as needed. Detailed instructions are included in the README.md file. If you need further explanation of the process just let me know.

The extension does not load! I have two folders (client - React project & server - Nodejs project). I use the extension without issues in one folder projects. I even add the different folders ${workspaceFolder}\client & ${workspaceFolder}\server, but the extension does not apper.

That is strange. Are you sure you have a package.json file in the root of your project? It will not load if it does not find one. Are you able to open the example files with the command Version Inc: Edit Example Files from the command palette? Just testing it now and I am having a problem getting the example files to update. I think vscode must of changed something regarding the global storage as I cannot get the example files to update even if I supply the full path in the settings file to the example files in global storage? It is working for me with files in the in the projects root folder and any folders below that though.
It must be late, lol. I just realized that the example files had the "Enable": true" setting set to false. They are updating correctly now. Check that setting in the settings file and let me know if it is set correctly.

Just thought more about what you said. Do you have a package.json in each of the folders, client and server? If that is the case you would still need one in the root folder as that is where it is searched for. If so you could just create a dummy package.json file in the root folder. All it would need to contain is the line "version": "1.0.0" that would give the extension something to update in the root. Then just add the other files to the settings file.

There is not a package.json in root folder. I created one and the extension appreared. I have two package.josn, one in client and one in server folder and these I need to update regularly.

You can add both to the settings file like this... (Add any other files you want to update as well)
[
	{
		"Filename": "package.json",
		"FileLocation": "client",
		"Enable": true,
		"RetainLine": true,
		"InsertBefore": "",
		"InsertAfter": "",
		"TrimTextStart": 0,
		"TrimTextEnd": 0
	},
	{
		"Filename": "package.json",
		"FileLocation": "server",
		"Enable": true,
		"RetainLine": true,
		"InsertBefore": "",
		"InsertAfter": "",
		"TrimTextStart": 0,
		"TrimTextEnd": 0
	}
]

You can also write "Filename": "client\\package.json" and "FileLocation": "" if you prefer.

I have already done this! But it seems not to increase the version value... If I put V-INV in my package.json files, but it works only the first time.

Yes this is a problem that needs to be fixed. If json supported comments this could be done easily by setting the "RetainLine": "true" and commenting the "version": "1.0.0" line but that won't work in a json file as it will create a duplicate line which will cause an error. I will have to add support for multiple package.json files in order to get this to work. I will look at it tomorrow as it is 4:30am here now and I need to get some sleep.

Just wanted to let you know that I am still working on this. It turned out to be much more involved than I thought it would be. Each package.json file has its own version independent of the others so that had to be taken into consideration. I have added a new setting allowing the user to define additional locations for for other package.json files, as many as you need. The default is set to client and server as that is very common. I currently have the updating for all package.json files in their defined locations working. If updating all, you will be prompted for increment type (patch, minor, major) for each defined package. The code still needs a bit of fine tuning and bug testing. I'll also have to do the same for decrementing the versions, although that is just basically just copying the changes from the incrementing code with some small changes, so that won't take as long.
Do note that the package.json in the workspace root is still a requirement. I'm not sure why you didn't have one in the root folder? I have never seen that in any project, for example most of vscode's language extensions use a client and server and all of them have package.json files in the root as well as the client and server folders. At any rate, I suggest you run npm init in the root folder to generate a basic package.json and set the version to what you want for the entire project. Then the client and server can have there own versions which makes sense as your usually only updating one at a time.
Anyhow, hopefully I will have this done soon, it is very close to being done now.

This has been completed and is working correctly. I just need to update the documentation before uploading the update. Should be available for download by tomorrow morning at the latest.

Uploaded as version 1.1.0 and is available to update within vscode. Let me know if this is working for you.

Great! It seems to work fine. Although, you need to have a dummy package.json on root folder to load the extension.
Many thanks

When you have more than one package.json files, it will be nice to have a "Maintain version number" option in menu (Patch/Minor/Major).

By "Maintain version number" do you mean you want one to not update? If so just you can also just hit 'Esc' to skip updating the currently selected file. You can also stop getting prompted for that particular file by removing its folder in the settings. Go to settings (ctrl+,) -> Extensions (on the left) and scroll down to 'Version INC' and delete from the list 'Package JSON Paths' the folder containing the package.json file you want to stop updating. You may want to do this using the 'Workspace' tab at the top left, as this will then only apply to that project. If you remove it from the 'User' settings tab, it will be removed from all projects.

By the way you actually should have a package.json in the root just to maintain the overall version of your project. The ones in client and server should have their own version numbers that are only updated when making changes to that component. They are the ones that will contain all your dependencies as well. The one in the root folder should basically contain just the projects name, description, and overall version number, no need to declare dependencies in that one. This is the most common way of doing this in this situation.

I should try to explain my reasoning for requiring a package file in the root folder. The main purpose of this extension is to update the version number in the package file and if it can't find it there is no reason for it to even load. The VS code activation event I use to load the extension "workspaceContains:package.json" actually expects a package.json file to be in the root, where everyone should have one, and if it is not there my extension just won't activate. The only way around that would be to manually search every sub-folder looking for package files but that would only be possible if I let the extension load first, then run the search code, and if no package is found then I would have to force my extension to unload. That is definitely not the right way to load an extension. The Marketplace would probably flag my extension as well for the unusual code as an extension, once loaded should only ever be disabled by the end user and never by the extension itself.
That was rather long but I hope it clears that up for you.


chore(events_list): 10166 - add description for active event and short state event

event description is now provided

so we show it for active event in full panel state

and for short state too

Codecov Report

Merging #318 (d4b1de9) into main (3527265) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #318      +/-   ##
==========================================
- Coverage   11.60%   11.60%   -0.01%
==========================================
  Files         537      537
  Lines       38604    38626      +22
  Branches     1127     1127
==========================================
  Hits         4481     4481
- Misses      33689    33711      +22
  Partials      434      434

Impacted Files
Coverage Δ

src/core/types/index.ts
0.00% <0.00%> (ø)

...ents_list/components/CurrentEvent/CurrentEvent.tsx
0.00% <0.00%> (ø)

...res/events_list/components/EventCard/EventCard.tsx
0.00% <0.00%> (ø)

...res/events_list/components/FullState/FullState.tsx
0.00% <0.00%> (ø)

...s/events_list/components/ShortState/ShortState.tsx
0.00% <0.00%> (ø)


Document Page - Folder should not copy in same folder

Describe the bug

when we copy folder and paste in same folder it should not copy or show confirmation message

https://www.berrycast.com/conversations/1e330a2f-a3f2-55d0-8ab2-b04f1cca92f5

This is required


Fix a typo in log message, and adjust code indentation

This PR fixes a typo in exception message and adjusts the code indentation, keeping tabs as used in other parts of the code.

thanks appreciate it.


Update Tuist and more dependencies using Carthage

Codecov Report

Merging #207 into master will increase coverage by 0.03%.
The diff coverage is 97.22%.

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
+ Coverage   57.61%   57.64%   +0.03%
==========================================
  Files         316      316
  Lines        6368     6375       +7
==========================================
+ Hits         3669     3675       +6
- Misses       2699     2700       +1

Impacted Files
Coverage Δ

...p/Shows/Progress/ShowsProgressViewController.swift
38.93% <0.00%> (-0.35%)
:arrow_down:

...ouchTrackerCore/Core/Bundle+CouchTrackerCore.swift
40.00% <ø> (ø)

...TrackerCore/MovieDetails/MovieDetailsService.swift
97.14% <100.00%> (+0.17%)
:arrow_up:

...rackerCore/Shows/Progress/ShowProgressFilter.swift
100.00% <100.00%> (ø)

...hTrackerCore/Shows/Progress/ShowProgressSort.swift
94.87% <100.00%> (+0.13%)
:arrow_up:

CouchTrackerSync/Models/Show.swift
100.00% <100.00%> (ø)

CouchTrackerSync/SyncFunctions.swift
98.71% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a96d7dd...2ba92a8. Read the comment docs.


Location Events log level should be the same

Expected Behavior
location cache events log level should be the same as location_found event log level
Actual Behavior
location_found log level is info,
location cache log level is debug
Steps to Reproduce
just run
Other Information
OS: Linux
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)
Python Version: (run 'python -V' and paste it here)

I will, respectfully, say that this is more of a design opinion than an actual issue. The location cache information is not entirely relevant to those simply wanting to start where they left off. Location_Found information is relevant to all users, hence the info categorization.
If you like seeing the location cache information you are free to leave the debug flag set to true but for those that find that information irrellevant, they can set debug to false and carry on without knowledge as to whether or not the location_cache was used.

This events are fired one after another only on start. if log level is info and location cache is true I see in log not actual position where I start.


[PL] Translation of Mix Tasks

Thank you @kdebowski!  @Koziolek can you take a peek at this?

No bugs.


Feature Request: an API to programmatically set language mode

The feature to set language mode programmatically would be beneficial for extensions like https://github.com/ahebrank/vscode-yaml2json that convert the current document to a different language.
Right now, the only semi-reliable way to do this seems to be renaming the opened file, changing its extension. But this does not really work if the file has not been saved yet, or if detection of the language in question does not depend of the extension.

Does this work for you - to change the active editor language mode:
await vscode.languages.setTextDocumentLanguage(vscode.window.activeTextEditor.document, 'css');

@ArturoDent my bad, I totally missed that in the docs


Post Login redirect

Expected Behavior
Once the user successfully login they should be redirected to the homepage.
Current Behavior
After a successful login dashboard is displayed, staying in the same page.
Failure Information (for bugs)
Once the user login they should be redirected to a page that provides the main contents of the website, in our case the homepage which will show all the news items.
Steps to Reproduce

Login using correct credentials.

I don't think that's the idea. @ivantha Could you please clarify?


[BUG] Editing a Community Does Not Work

Bug
After summitting the form to update a community, you get redirected to the home page and nothing gets updated.
Expected Behavior
You should get redirected to the communities page and the information should be updated.

Fixed in the latest release


[WIP] add gitignore and license files

:warning: This is a work in progress do not merge yet :warning:
TODO

[x] Gitignore
[ ] LICENSE

@fakeusername1234321 - tag someone to look at this

also cc #2


[Event Request] table 83 "Item Journal Line" OnBeforeCheckNewTrackingIfRequired

Can you add event CheckNewTrackingIfRequired to table 83 "Item Journal Line" to be able to override the check.
    procedure CheckNewTrackingIfRequired(ItemTrackingSetup: Record "Item Tracking Setup")
    begin
        //>>>>>>>>>>>>>>
        OnBeforeCheckNewTrackingIfRequired(Rec, ItemTrackingSetup, IsHandled);
        if IsHandled then
            exit;
        //<<<<<<<<<<<<<<
        if ItemTrackingSetup."Serial No. Required" then
            TestField("New Serial No.");
        if ItemTrackingSetup."Lot No. Required" then
            TestField("New Lot No.");

        OnAfterCheckNewTrackingIfRequired(Rec, ItemTrackingSetup);
    end;

Hello @auzhhv
Unfortunately, we won't implement this event.
It should be done on higher level, where this function is called.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: .


Microsoft.SharePoint.MigrationTool.PowerShell does not migrate files without major version

I use Microsoft.SharePoint.MigrationTool.PowerShell to migrate 2013 Library to SPO folder in a library and found out that all the files that did not have a major version, did not migrate.
Is there a flag I should use to make it migrate?
Thank you

Hello @Ofer-Gal
I tried to reproduce to see if we need to adjust any doc, but couldn't, can you detail the options used in the Register-SPMTMigration? thanks

Register-SPMTMigration -SPOCredential $Global:SPOCredential -Force -PreserveUserPermissionsForSharePointSource $false
Is there a settings to force all the files? I did not find it
Thank you

Is there any entry in the log file about those files not being migrated?
Also be sure to be running the latest version of the spmt module.

@get-itips Thank you very much for the contribution and digging deep into it.  @Ofer-Gal We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


add default values to Dockerfile to simplify nldi-db configuration

This is part one.  Part two will be in nldi-services, where a number of configuration settings will be removed (relying on the new defaults here).

This pull request passes locally.  It's failing here because the demoDB never rebuilt on dockerhub yesterday for some reason.  I'm currently asking how to resolve the dockerhub situation.

@kkehl-usgs -- noticing this PR is lingering. Should it get some attention?

Closing because this is a very low-priority "nice to have" and it is blocked by unrelated problems regarding the new artifactory.  I'll update the NHGF-11 JIRA ticket with comments.


React Native is not accepting project.android.mainFilePath as an acceptable configuration

Description
v 0.68.2 accepts our react-native.config.js which is as follows:
module.exports = {
  project: {
    ios: {},
    android: {
      mainFilePath: 'src/main/java/com/myantskiin/MainApplication.kt'
    }
  },
  assets: ['./src/assets/fonts/'],
  dependencies: {
  },
}

However, v 0.69 throws the following error:
error Failed to load configuration of your project. Config Validation Error: "project.android.mainFilePath" is not allowed     at readConfigFromDisk (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli-config/build/readConfigFromDisk.js:48:11)     at loadConfig (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli-config/build/loadConfig.js:69:65)     at setupAndRun (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli/build/index.js:250:45)     at Object.run (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli/build/index.js:212:11)     at Object.<anonymous> (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/cli.js:16:7)     at Module._compile (internal/modules/cjs/loader.js:1085:14)     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)     at Module.load (internal/modules/cjs/loader.js:950:32)     at Function.Module._load (internal/modules/cjs/loader.js:790:12)     at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:75:12) info Run CLI with --verbose flag for more details.
Version
0.69.0
Output of npx react-native info
error Failed to load configuration of your project.
Config Validation Error: "project.android.mainFilePath" is not allowed
at readConfigFromDisk (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli-config/build/readConfigFromDisk.js:48:11)
at loadConfig (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli-config/build/loadConfig.js:69:65)
at setupAndRun (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli/build/index.js:250:45)
at Object.run (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/node_modules/@react-native-community/cli/build/index.js:212:11)
at Object. (/Users/distiller/.nvm/versions/node/v14.19.3/lib/node_modules/react-native/cli.js:16:7)
at Module._compile (internal/modules/cjs/loader.js:1085:14)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
at Module.load (internal/modules/cjs/loader.js:950:32)
at Function.Module._load (internal/modules/cjs/loader.js:790:12)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:75:12)
info Run CLI with --verbose flag for more details.
Steps to reproduce
Just use the react-native.config.js with mainFilePath and it does not accept it as a valid configuration.
Snack, code example, screenshot, or link to a repository
module.exports = {
project: {
ios: {},
android: {
mainFilePath: 'src/main/java/com/myantskiin/MainApplication.kt'
}
},
assets: ['./src/assets/fonts/'],
dependencies: {
},
}

Just use the react-native.config.js with mainFilePath and it does not accept it as a valid configuration.

This file is handled by the React Native CLI. Please open the same issue under: https://github.com/react-native-community/cli


Remove periods from list in about-mandatory-two-factor-authentication#about-eligibility-for-mandatory-2fa

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/authentication/securing-your-account-with-two-factor-authentication-2fa/about-mandatory-two-factor-authentication#about-eligibility-for-mandatory-2fa
What part(s) of the article would you like to see updated?

 Eligible actions include:

Publishing an app or action for others.
Creating a release for your repository.
Contributing to specific high-importance repositories, such as the projects tracked by the Open Source Security Foundation.
Being an administrator of a high-importance repository.
Being an organization owner for an organization containing repositories or other users.
Being an enterprise administrator.

Periods need to be removed from each list line, as each line is not a complete sentence (per our style guide)
Additional information
No response

Closing as the following PR will address this issue once merged (when repo freeze ends on 9/16) - https://github.com/github/docs/pull/34524


Don't inherit --inspect flag from the host process

To avoid conflicting ports like https://github.com/zeit/next.js/issues/11821
fork-ts-checker-webpack-plugin passes all env variables from the host process to its subprocess. If the host process was started with NODE_OPTIONS containing an --inspect flag, it will try to start a debugger on the same port as the host process is trying to start a debugger. One of the processes will fail to start since the port is already in use. This code filters out this CLI flag from the NODE_OPTIONS env variable if it exists for the host process.

Before this is merged, we should probably look into https://github.com/zeit/next.js/pull/12555 as I think it applies to this PR as well

@Janpot
Could you check if this issue was maybe solved with the v5.0.0-alpha.1 release? :)

I don't think it's solved, but next doesn't enable this plugin anymore, so it doesn't really matter to me anymore


language cannot be changed

Hey, 4minitz is great but I cannot manage to change the UI language. I tried a local docker on linux and a docker on a synology NAS (http://<IP_ADDRESS>:3100). No matter which platform or browser I use, the language is always english and a cannot find a setting to change this. Is there a solution for this?

Hey, great you like 4Mintz. The current stable version v1.5.2 is only available in English.
We are preparing a version 2.0 (that will still take some time) with multi-language support.
You may check here on translation progress:
https://crowdin.com/project/4minitz
(The first percentage value is state of machine translation, the second percentage number is state of native speaker review and corrections).
If you want to try out the current development state, use the development test server here:
https://develop.4minitz.com/
Or install the docker "unstable" image.
Feel free to re-open this issue if you have language switching problems with the current development version.

Thank you @derwok for your answer. I managed to set up version 2.0 though I really struggled with mongodb ;).
The login screen does show up in the selected language, but the toolbar and everything when I am logged in, stays in english. Do you have any idea what may cause this issue?

Pre-Login we cannot store your language preference.
Once you logged in you can change language via User Menu / Set Language.
Then we'll store it for the future for this user (when user is logged in).
Best way is to set in your browser settings your generally preferred
language. This browser setting we'll use first if user has not specified
anything else.
WOK
On 27.08.20 10:31, MrCardigan wrote:

Thank you @derwok https://github.com/derwok for your answer. I
managed to set up version 2.0 though I really struggled with mongodb ;).
The login screen does show up in the selected language, but the
toolbar and everything when I am logged in, stays in english. Do you
have any idea what may cause this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/4minitz/4minitz/issues/561#issuecomment-681782698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBQVYRZS6RNXZYE7ZY5ZD3SCYKWNANCNFSM4QHHMEHQ.

Hi @MrCardigan,
have you configured the ROOT_URL within your settings.json correctly? Your browser will try to download the translations from the given ROOT_URL and hence fails if ROOT_URL is not reachable.
Best wishes,
Eduard

Thank you @derwok & @itrich ,
I'm new to git and did not expect more answers, if I delete my comment :). In fact I just recognized the ROOT_URL setting after I commented again 🙈 . That solved the Problem


In demo app, autocompletion and random jump stopped working

Try entering address in Route tab or Business name tab
Try random jump in route tab.
Failed to load resource: the server responded with a status of 404 ()
leaflet.js:5 Deprecated include of L.Mixin.Events: this property will be removed in future releases, please inherit from L.Evented instead. Error
at y (https://unpkg.com/leaflet@1.2.0/dist/leaflet.js:5:2048)
at Function.v.extend (https://unpkg.com/leaflet@1.2.0/dist/leaflet.js:5:14706)
at https://bcgov.github.io/ols-devkit/ols-demo/leaflet-layerjson.js:4:30
at https://bcgov.github.io/ols-devkit/ols-demo/leaflet-layerjson.js:390:4
y @ leaflet.js:5
Sortable.js:1 Failed to load resource: the server responded with a status of 404 ()
index.html:2084 Uncaught ReferenceError: Sortable is not defined
at index.html:2084

This was already fixed and delivered in PR #169. I think it should be marked as done.

Verified in test


OCPBUGS-9960: Add disable operand delete annotation

Description
Add annotation to disable operand delete option on console. This acts as a check, to ensure deletion is done manually.
cc: @xenolinux docs updated required for manual deletion

/cc @swghosh

/retest-required

/lgtm
/cc @davemulford
for px-approved
/cc @xenolinux
for docs-approved
/jira cc-qa
for qe-approved

/jira refresh

@thejasn is there anything in the UI that instructs the customer to manually delete the operands before uninstalling the operator?

There is a follow up docs task which will add steps to the uninstall section of the operator documentation.

/label qe-approved
Thanks!


concatMap with deferreds / chaining promises

Any ideas on how I would do the equivalent of this in most?
http://jsbin.com/kodoni/2/edit?js,console
var source = Rx.Observable.of(1,2,3,4)
  .concatMap(function(x) {
    return Rx.Observable.defer(function() {
      return new Promise(function(resolve, reject) {
        setTimeout(function() {
          resolve(x);
        }, 1000);
      });
    });
  })
  .subscribe(function(i) {
    console.log(i);
  });

Hmmm, looks kind of complex for what it seems to do when I run it. Is the intention to print a counter that increments at 1 second intervals?

Given a stream, I would like to run async operations on each value, but only one at a time. Similar to async's .series
Scenario: a stream of urls that need to be downloaded one at a time.

Ideally concatMap would process a series of promises in sequence:
most.from([url1, url2, url3])
  .concatMap(function(url) {
    return doSomethingAsyncWith(url) // return a promise
  })
  .then(console.log.bind(console))

Aren't you simply missing the consumer part? such as forEach
most.from([url1, url2, url3])
  .concatMap(function(url) {
    return most.fromPromise(doSomethingAsyncWith(url))
  })
  .forEach(console.log.bind(console))  //-> url1.data, url2.data, url3.data in that order
  .then(function(){
     console.log('done');
  })

the forEach returns a promise which we can use signal that all of the promises have been resolved.

While that outputs the resolved values in sequence, that doesn't execute the promises themselves in sequence.

Ah, cool, thanks for clarifying.  Here are two ways that came to mind.  The first is more declarative, and the second a bit more imperative.  In my mind, the first represents the goal more clearly: to accumulate a sequence of urls into a sequence of future values concatenated in time.  The skip(1) is slightly unfortunate.
most.from([url1, url2, url3])
	.scan((queue, url) => queue.then(() =>  doSomethingAsyncWithUrl(url)),
		Promise.resolve())
	.skip(1)
	.await()
	.observe(content => console.log(content));

most.from([url1, url2, url3])
	.concatMap(url => {
		return most.create((add, end) => {
			add(doSomethingAsyncWithUrl(url));
			end();
		})
		.await();
	})
	.observe(content => console.log(content));

By design, RxJS considers promises to be another kind of observable.  In contrast, most.js keeps Promise and Stream separate by design, so operations don't implicitly turn promises into event streams.  The reason is that we feel this gives you more control, by using await() for example, over when promises' ultimate value gets merged into a stream.

Or by unfolding:
most.unfold(function(urls) {
	return urls.length === 0
		? { done: true }
		: fetch(urls[0]).then(content => { value: content, seed: urls.slice(1) })
}, [url1, url2, url3])
	.observe(console.log);

@aaronshaf Did the examples above help?

I haven't tried the latest yet. I will and report back. I am trying to prevent the premature creation of promises.

I am having a tough time getting this to work in a browser (bundled with webpack):
https://gist.github.com/aaronshaf/6b20f1e4c0bd3d78b704
Chrome complains about an "Illegal invocation."
Works fine when run directly with babel-node

@aaronshaf Sorry, my bad.  You can't call console.log as a function in most envs.  It works in node/io.js, but not in many browsers.  Here's a fork of the gist

Ah! A simple fix!
This works wonderfully, and it is exactly the kind of thing I was looking for. Thank you!

Yay! :+1:

Hey all, I had a similar but slightly different issue that this ticket helped me through- rather than trying to run fetch() multiple times on a list of urls, I was getting a stream of domainNames, and had a function that returned a promise with a list of records in the name. I was trying to figure out how to flatly take the Array-fulfilling promise results to a new stream.
I used @briancavalier's suggestion of unfolding to get by this, in slightly different form. It took me a pass or three to make right/not-ugly, so I built a library for the final version- most-promise-spread.
From the example there, I got mostPromiseSpread(Promise.resolve([.forEach(x => console.log(x*2)). For domainNames/records, it looks something like:
domainNames.concatMap(n => mostPromiseSpread(lookupRecords(n)).


docs: Chinese version of Clusternet shadow APIs

Signed-off-by: mango<EMAIL_ADDRESS>Fixes #48

@mangoGoForward Good work. Thanks for the contribution.


Update README.md

Added some changes in the readme file like some information about the camunda-modeler.

Which issue does this PR address?
Closes #
Acceptance Criteria

[ ] Corresponds to the concept
[ ] Corresponds to the design

Definition of Done

[ ] corresponds to the design principles
[ ] corresponds to the code standards
[ ] passes Continuous Integration checks
[ ] available as feature branch on GitHub

[ ] contains the cleaned up commit history
[ ] commit messages satisfy our commit message guidelines
[ ] a single commit closes the issue via Closes #issuenr

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add Gm17 arkworks backend

In the interest of not supporting a backend which may not be used, I'll tentatively close this issue


When a submission for a Project is Exported, change the Project status to Submitted

As a data wrangler and stakeholder, I want to have a correct view of how many projects have been Submitted over a certain period of time.
This Miro board prepared by @ESapenaVentura @Mil-m & @amnonkhen contains the project status transition chart and outlines the required transitions to be automated.
Current situation:
A data wrangler would forget to change the status of a project to Submitted, and then the number of projects Submitted over a period of time or in a release would not be correct.
NB:
if a new submission appears per project, change status back to In progress, and then do the above when Complete.
Note - this scneario is removed from the initial scope, because an update to a project that is already in the dcp is, in theory, in both statuses at the same time. It is "in progress" but also "exported". This requires further discussion and refinement.
Acceptance Criteria
see board

To discuss the options at the design meeting. What happens with new submissions.

This sequence diagram shows the sequence of events for creating a new submission, the relevant involved components and the location where the new functionality will be would be implemented.
sequenceDiagram
    User ->> UI: create submission
    UI ->> Broker: REST call: POST /api_upload
    Broker ->> Core: REST call create submission envelope
    note right of Broker: POST '/submissionEnvelopes'
    Core -->> Broker: submission
    Broker ->> Core: associate submission with project (REST)
    note right of Broker: POST /projects/<project-uuid>/submissionEnvelopes <br> body '<ingest-api-url>/submissionEnvelopes/<submission-id>'
    Core ->> Core: update project status
    note right of Core: This needs to be developed

See draft PR with test: ebi-ait/ingest-core#141

@Mil-m to check and fix the test for statuses

sequenceDiagram
   actor Wrangler
   participant Core
   participant Exporter
   participant Queue
   participant StateTracker

   Wrangler ->> Core: export
   Core ->> Exporter: export jobs
   Exporter ->> Core: Job Completed
   Exporter ->> Unknown: job completed
   Unknown ->> StateTracker: submission exported
StateTracker ->>Core: submission exported
  Core ->> Core: update submission state to exported

this was tested
now waiting for #850 to fix a github issue

Once testing is complete:
This ingest-core pipeline and this ingest-exporter pipeline can be released to production.

There is a bug relevant to this feature, which is still being investigated by @amnonkhen

Let's have this tested/confirmed by a wrangler, and see if this works correctly. @ESapenaVentura


add options.

this option need because other language support.(like korean.)
it can be found in isProfane function when I change regex option into  "/[^a-zA-z0-9가-힣$|@]|^/g"  but it is not replace in replaceWord function cuz that function uses fixed regex '/\w/g'.
so, I added few line for anyone can change replace regex like /[A-Za-z0-9가-힣_]/g .

Looks good. All tests are passing. Its in the latest release.
Thanks!


Events to hook into for logging

Thanks for this project.
I was just wondering if they is a way to hook into events for when the rate limiting starts and stops?  What I would like to be able to do in my application is log when the code has been paused due to ratelimiting kicking in and then log when the code has resumed.

This is a good suggestion, I am thinking of providing an alternative implementation with events based on IObservable pattern.  I will update this thread later on with more details.

Hey @David-Desmaisons! I don't suppose you have any further updates to this? We're using RateLimiter in our project as it hugely simplifies things and avoids having us roll out our own rate limiting solution.
We'd love some sort of events or statistics on rate limiters that we can graph the output of, to track how often we're hitting our rate limiters or similar across the platform!


Conditional types don't narrow the type for Mapped types

cc @ahejlsberg
TypeScript Version:  master

Search Terms:
Code
type A<T> = T extends string ? { [P in T]: void; } : T;
type B<T> = string extends T ? { [P in T]: void; } : T;

Expected behavior:
pass
Actual behavior:
$ node built/local/tsc.js --noEmit index.ts
index.ts(1,40): error TS2322: Type 'T' is not assignable to type 'string'.
index.ts(2,40): error TS2322: Type 'T' is not assignable to type 'string'.

Playground Link:
Related Issues:

cc @ahejlsberg

Now fixed in #21744. Note that B<T> still produces an error as expected.

Thanks for your quick work!


#/definitions/article/links/1/href: failed schema

Hi,
I'm getting a:
unverified_schema.json: #/definitions/article/links/1/href: failed schema #/definitions/resourceLink/properties/href: /articles/ does not match /^(\/((?!-|_)[a-z0-9_\-]+(?<!-|_)|\{\(.*\)\}))+$/.

Which is true, I have a:
{
    "description": "List some articles.",
    "href": "/articles/",
    "method": "GET",
    "rel": "instances",
    "targetSchema": {
        "$ref": "/schemata/article#/definitions/list_of_article"
    },
    "title": "Article list"
}

Which looks right to me, listing articles (a collection, a direcrory of articles) is at "/articles/", not at "/article" (which means a file, which is different from a collection of files), linking to articles with URLs like "/articles/1234" ("files").
Why does URI with a trailing are forbidden ? They look right to me, can you allow it in the URL regex ?

I think this may just be a consistency thing. Having an href there of either /articles/ or /articles should be equivalent in usage, so just trying to make sure it is consistently the shorter/simpler version doesn't seem so bad. Does that help/make sense?

Hi,

Having an href there of either /articles/ or /articles should be equivalent in usage

If equivalent in usage, they still can be considered semantically different:
I like to see a REST API as as simple HTTP server serving files, so, for me: "/articles/" is a directory (a collection), and "/article" is a file which does not exists. "/articles/1" is a file which exists, and "/articles/1/" is a directory which does not exists.
From my very personal point of view, a REST API is not just bytes flowing on networks (the usage), it should me meaningfull, it should expose resouces in an expressive and semantical way, and here, trailing slash helps me being explicit and expressive (for the users).
I know there's an infinitely long debate here, let's not start another (Woops, I started, I'm very sorry, and not proud):

Having a trailing slash like "/article/" permits relative URI like "1" to resolve to "/articles/1/" which is NOT to case with "/article" in which a URI like "1" resolve to "/1" which does not exists.
https://en.wikipedia.org/wiki/Representational_state_transfer#Example examples with "/" being collections and without "/" being collection members
Some blog (yeark) speaking about trailing slashes : https://cdivilly.wordpress.com/2014/03/11/why-trailing-slashes-on-uris-are-important/
I don't want to redirect "not slash" to "slash": that's a useless redirection the developper could have fixed upfront hitting a violent 404, avoiding de facto a huge number of very useless 301/302 in production (useless round-trips, useless increased load times ....)

But I know it's controversial:

https://github.com/nicolaiarocci/eve/issues/118
I don't remember seeing any reference to this in the Fielding thesis
http://masnun.com/2013/09/18/django-rest-framework-angularjs-resource-trailing-slash-problem.html

Very controversial:

if /users/ is a collection of users (a directory)
and /users/1 is a user (a file)
but /users/1/posts/ is a collection of the 1st user posts
WTF did I just said ? A file having a directory in it ?

Yet ... the API I'm writing HyperSchema for already exists, obviously can't be changed, is already documented and used with the philosophy "trailing slash means collection". Does the existence of a debate, even with more people against it that agreeing on it, should prohibe us to documents this API ?
So if you're definitely on the "no trailing /" side, a warning should be more appropriate than a "throw exception and exit" here ?

Fair point, I'd never really thought about/worried about it very deeply to be honest (hence my comment about interchangeability), for me they were roughly equivalent and in cases where there are equivalences I like to just pick one and stick with it. That said, I still don't feel particularly strongly about this one way or another, I'm amenable to allowing the trailing slash, would you be up for making a pull request to loosen the expectation? Thanks!


Fix UI Feature section of home page

Description
The feature section having a lot of top and bottom margin and not proper responsive.
Screenshots

Can fix this issue.
Assignee: Please Assign this to me.
Additional information
No response
What browser are you seeing the problem on?
No response
Checklist

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

Assigning to you, @VishalGaurav01.


how to upload content without a local path

I want to upload the content directly to iCloud, something like upload(cloudPath, content, { onProgress }),  basically a version of writeFile + onProgress, without the need to create a temporary file, but also support onProgress callback.
I believe this is a very common case, is there a method to do it? Thanks

Currently you need to do as you said.
I will add onProgress support for writeFile next version, and thanks for your suggestion.

发件人: Guten Ye @.>
发送时间: Thursday, March 2, 2023 5:01:04 PM
收件人: XHMM/react-native-cloud-store @.>
抄送: Subscribed @.***>
主题: [XHMM/react-native-cloud-store] how to upload content without a local path (Issue #14)
I want to upload the content directly to iCloud, something like upload(cloudPath, content, { onProgress }), basically a version of writeFile + onProgress, without the need to create a temporary file, but also support onProgress callback.
I believe this is a very common case, is there a method to do it? Thanks
―
Reply to this email directly, view it on GitHubhttps://github.com/XHMM/react-native-cloud-store/issues/14, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF5DMWTMCNFATCD3NJFXEFDW2BOVBANCNFSM6AAAAAAVNC3T6Q.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

supported in v0.10.0


Subject extra space leads to irrelevant references may not be empty error

Current Behavior

N
"Input"
Commitlint summary
ok or not

1
"fix: #1 fix "  (with extra end space)
1 problems, references may not be empty
❓

2
"fix: #1 fix"
0 problems
✔

Expected Behavior
"fix: #1 fix " should be linted with no errors or should have another error message.
Affected packages

<EMAIL_ADDRESS>
Possible Solution

Steps to Reproduce (for bugs)

Create commitlint.config.js with content below
Execute

$ echo "fix: #1 fix " | node_modules/.bin/commitlint
⧗   input:
fix: #1 fix

✖   references may not be empty [references-empty]
✖   found 1 problems, 0 warnings

Observed error references may not be empty while the reference is here ❓

Successful example
Without extra space in the subject works as expected ✔
$ echo "fix: #1 fix" | node_modules/.bin/commitlint
⧗   input: fix: #1 fix
✔   found 0 problems, 0 warnings

commitlint.config.js
module.exports = {
  'rules': {
    'references-empty': [2, 'never'],
  }
};

Context
Commit messages verification.
Your Environment

Executable
Version

commitlint --version
7.0.0

git --version
git version 2.17.1.windows.2

node --version
v8.11.1

Closing as that behaviour is per reference actions that looks for keywords like fix, fixes, fixed, close and there expect fix #1 order.
It can be disabled with overidding referenceActions with smth reasonable.
module.exports = {
  'rules': {
    'references-empty': [2, 'never'],
    'referenceActions': '__declare_no_actions'
  }
};


schema with [name: str]: Test = { name = name } is not being populated in the schema

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)
It looks like the types captured for TestMap in here, cannot capture Properties.

package main

import (
	"bytes"
	"fmt"
	"os"

	"kcl-lang.io/kcl-go/pkg/tools/gen"
)

const code = `
schema Main:
	maps: TestMap

schema Test:
	name: str
	surname: str

schema TestMap:
	[name: str]: Test = {name = name}
`

func main() {
	var buf bytes.Buffer
	err := gen.GenGo(&buf, fmt.Sprintf("test.k"), code, nil)
	if err != nil {
		fmt.Println(err.Error())
		os.Exit(1)
	}

	fmt.Println(buf.String())
}

2. What did you expect to see? (Required)
type TestMap map[string]Test

type Test struct {
    name string `kcl:"name=name,type=str"`       // kcl-type: str
    surname string `kcl:"name=surname,type=str"` // kcl-type: str
}

type Main struct {
    maps *TestMap `kcl:"name=maps,type=TestMap"` // kcl-type: TestMap
}

3. What did you see instead (Required)
type TestMap struct {
}

type Test struct {
    name string `kcl:"name=name,type=str"`       // kcl-type: str
    surname string `kcl:"name=surname,type=str"` // kcl-type: str
}

type Main struct {
    maps *TestMap `kcl:"name=maps,type=TestMap"` // kcl-type: TestMap
}

4. What is your KCL components version? (Required)

0.10.10-darwin-arm64

/cc @Peefy


Possible to save file multiple times leads to race condition

When auto save is disabled and you work against a slow network share where save takes longer, it is possible to save a file multiple times with different contents leading to bad race conditions where you sporadically see warnings about the contents on disk being newer.
Part of https://github.com/Microsoft/vscode/issues/13665

To verify:

connect to a super slow network share (all our internal shares are a good candidate for this)
disable auto save
edit a file and save
=> this should take quite some time
edit a file and save, then continue editing and save
=> basically try to cause many saves (you need to cause changes, otherwise the save request is ignored)

You should never see a save error and the save should eventually complete.


[dev] Update Jest SVG renderer

Updates the SVG renderer.
In looking at the lockfile: I feel like esdoc was removed from a dependency (might have been a one of my danger plugins,  I remember doing something like this) and then this wasn't yarn remove'd - resulting in the new lockfile.

This is awesomeeeeee

Fails

:no_entry_sign:

No CHANGELOG added.

:no_entry_sign:

Please assign someone to merge this PR, and optionally include people who should review.

  Generated by :no_entry_sign: dangerJS


Could not find dependency

Overview (Required)

I bought a new MacBook Pro. I can't find the dependency. Probably this is related with https://github.com/DroidKaigi/conference-app-2018/pull/676 🤔

Could not find android.arch.persistence.room:compiler:1.0.0.
Searched in the following locations:
    file:/Users/takahirom/Library/Android/sdk/extras/m2repository/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.pom
    file:/Users/takahirom/Library/Android/sdk/extras/m2repository/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.jar
    file:/Users/takahirom/Library/Android/sdk/extras/google/m2repository/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.pom
    file:/Users/takahirom/Library/Android/sdk/extras/google/m2repository/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.jar
    file:/Users/takahirom/Library/Android/sdk/extras/android/m2repository/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.pom
    file:/Users/takahirom/Library/Android/sdk/extras/android/m2repository/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.jar
    https://repo.maven.apache.org/maven2/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.pom
    https://repo.maven.apache.org/maven2/android/arch/persistence/room/compiler/1.0.0/compiler-1.0.0.jar
Required by:
    project :app

I fixed it.
But I don't know why 🤦♂
I tried this. Please refer if you can not build

Refresh

Set variant

Sync project with Gradle Files


A Unity Blockchain Game Course using Flow

A Unity Blockchain Game Course using Flow
Educational material
Description
I’m proposing to create a course that covers creating a game in Unity and integrating it with the Flow blockchain. Due to the recent release of the Flow SDK for Unity, a comprehensive course on how to use this SDK would be very beneficial to the developer ecosystem. Due to the fact that Flow was explicitly designed to support games, its strong support for NFTs and upgradable smart contracts, I feel Flow is well positioned to be the number one solution in this space. Unity is already the preferred game engine for these kinds of projects however there are remarkably few resources available showing how to achieve this kind of integration. Having over 93,000 students on Udemy and over 160,000 course enrolments, I am uniquely positioned to introduce the Flow blockchain to a large number of new game developers.
The game for the course will be an enhanced version of Heroes of the Flow, a winner in the first Flow hackathon. Extra features, such as hero levelling and multiplayer, will showcase the power of the Flow blockchain. Additionally, by working closely with the T.E.A. team (A Dapper Labs Studio), and with their ongoing support, the course will be the definitive guide for creating a fully featured blockchain game in Unity.
Problem statement
There is a severe shortage of information on how to integrate Flow with Unity and what is out there lacks visibility. The target audience for the course is Unity game developers and the evidence for the need is the high demand for both block chain and game dev courses but the lack of courses that combine these two crucial elements in an easy to understand format.
Proposed solution
A course that covers creating a game in Unity that integrates the Flow blockchain via the Flow SDK for Unity.
Impact
This project will dramatically boost the number of game developers entering the Flow ecosystem and ensure that they have a thorough understanding of how to integrate Flow into their Unity game.
Milestones and funding

Note: Please consider adoption and/or maintenance milestones at the end of your project.

Milestone
Deliverables
Timeline
Risks
USD proposal

1 - Course Start
Course Plan, Dev & Art commissioned
1 week
-
20000

2 - Game / Art 30% Complete
Multiplayer / Hero Levelling Design / On-chain enemy stats / 30% Game Art Assets delivered / UI/UX 30%
1 month
-
5000

3 - Game / Art 60% Complete
Multiplayer / Hero Levelling Design / On-chain enemy stats / 60% Game Art Assets delivered / UI/UX 60%
1 month
-
5000

4 - Game / Art 100% Complete
Multiplayer / Hero Levelling Design / On-chain enemy stats / 100% Game Art Assets delivered / UI/UX Complete
1 month
-
5000

5 - Course Video content 50% Complete
Final Game Testing, Bug Fixing etc / Course Video content 50% Complete
1 month
-
5000

6 - Course Released
Course Video content 100% Complete
1 month
-
5000

7 - Adoption
Growing student enrolment to 20,000 students
2 months
-
4000

Total funding proposed: 49000 USD
Team

Name
Role
Bio
Contact

Morgan Page
Course Creator
https://www.linkedin.com/in/morganjpage/
<EMAIL_ADDRESS>

Simone Silva
Artist
https://www.linkedin.com/in/simone-silva-70178524/
<EMAIL_ADDRESS>

Anna Stoilova
UI / UX
https://www.linkedin.com/in/annastoilova/
<EMAIL_ADDRESS>

Hey Morgan,
I think this would be a great tutorial series to have. I don't know if you have an existing game or not, but I think you should consider adding time in to develop out the game first and add the hours and budget in for it.
We made this tutorial for the Avalanche blockchain. A very rudimentary game with very simple mechanics. https://github.com/CodeLinkIO/MoveAndWhap
It took about 2 months to pump out between 2 people, 1 smart contract and backend, the other frontend and wallet integration. That is not counting art time because we just used free assets. Additionally, a small game studio in Los Angeles usually spends about 10k-15k USD a month per employee. So, I would consider adding some of those development costs and times in there.
Good luck though, I am pretty much a self taught programmer who learned from people that make content like you. So my whole career is thanks to this type of work. Hope you get some interest from this.

hi @morganpage, thanks for your submission! We'd love to connect with you to discuss this proposal in more detail.
Please reach out to me directly on Discord to get the conversation going: alx-flw.find#6198.

Thanks @Codie-Petersen for your comment, I will definitely check out your Avalanche tutorial.
You are correct that creating a blockchain game AND doing a course covering its creation is a huge undertaking. Luckily the costs only need to cover my time and the artists.
We actually did a 7 day hackathon, Heroes of the Flow, which gives me confidence that such a course is achievable within the specified budget. Of course a larger budget would help facilitate a more comprehensive course so I'm sure this will be part of the ongoing discussions with the Flow team.
Really appreciate your support!

@morganpage Nice. Well good luck with the talks man. I'll keep an eye out for your content.

Hey @morganpage wanted to followup on this to se what the status was on Milestone 7? Considering closing this grant because I haven't seen signs of activity.


New version: Brave.Brave.Nightly version <IP_ADDRESS>

This pull request is automatically generated by 🥟 Dumplings
Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


ipv4_network_definition is not a managed PL

Hello,
From this module documentation:

The variable var.ipv4_network_definition is an attribute to configure a supernet (IPv4 CIDR block) or managed prefix list of your network/VPCs in AWS.

But it seems that the module consider var.ipv4_network_definition as an AWS Prefix List: from modules/egress_vpc:
data "aws_prefix_list" "ipv4_network_definition" {
  count = startswith(coalesce(var.ipv4_network_definition, " "), "pl-") ? 1 : 0

  prefix_list_id = var.ipv4_network_definition
}

Thus it can't find a customer managed Prefix Lists.
BR,
Damien.

Hi!
Thanks for your comment.
I don't fully understand your question here. What are you trying to achieve and what's the error?
That variable expects either a list of CIDRs or a customer-managed prefix list (in case you don't have a contiguous block that you define in a supernet). This variable is used in the corresponding VPCs (mainly the egress/ingress ones) to route to the Core Network.
Regards,
Pablo

Hi Pablo,
I used a workaround but from what I remember, if I put a Customer-managed prefix list in ipv4_network_definition I get an error.
It seems due to the fact that the "egress_vpc" module looks up this var from the aws_prefix_list Terraform resource. I guess that it should also try to find this prefix list from aws_ec2_managed_prefix_list Terraform resource. Terraform publishes different resources for Customer-managed prefix lists and for AWS-managed prefix lists.
Hope this helps.
BR, Damien.


Is it possible for Copy to loop forever?

Here is some code from https://github.com/tokio-rs/tokio-io/blob/master/src/copy.rs#L73-L78:
while self.pos < self.cap {
    let writer = self.writer.as_mut().unwrap();
    let i = try_nb!(writer.write(&self.buf[self.pos..self.cap]));
    self.pos += i;
    self.amt += i as u64;
}

Is it possible for try_nb!(writer.write(&self.buf[self.pos..self.cap])) to return 0? If so, wouldn’t it cause the code to loop forever?

@EFanZh ah yes indeed! I think that this should explicitly handle the 0-write case and return an error. Do you want to send a PR?

@alexcrichton Sure. A pull request is submitted: #57.

Fixed in #57


Documentation

I'm a bit new to Unreal and would like to implement GOAP, but I don't quite have an idea of where to start with this plugin. I have reviewed your GOAPdemo but I still don't know exactly what I need to do to get even the most basic prototype working as there is no documentation anywhere.

If you're new make sure you click the Class Defaults button in the GOAPAIController and GOAPAction blueprints.


Future.value doc could be more helpful

URL: https://api.dartlang.org/apidocs/channels/stable/dartdoc-viewer/dart-async.Future#id_Future-value
the doc for Future.value says
"A future whose value is available in the next event-loop iteration."
if the meaning of
"available at time t" is that
"the 'then' and 'whenComplete' callbacks will be called at some moment <= t",
then this is correct, but not as helpful as it could be, especially when writing a test.
e.g. i can't tell from these docs that
new Future(() => 1).then(foo) and
new Future.value(1).then(bar)
are completely different.  the former schedules foo on the event queue while the latter schedules bar on the microtask queue.  so if i want to test that some code can tolerate a value not being ready until the next event loop iteration, i can't use Future.value, i have to use the normal Future constructor.
e.g. demonstrating that Future.value schedules handling completion on the microtask queue:
  print('start');
  scheduleMicrotask(() {print('microtask 1');});
  new Future.value(1).then((x) {print('value completed $x');});
  scheduleMicrotask(() {print('microtask 2');});
  new Future(() {
    print('normal future running');
    scheduleMicrotask(() {print('microtask 4');});
    return 1;
  }).then((x) {print('normal future completed $x');});
  scheduleMicrotask(() {print('microtask 3');});
  print('end');
prints
  start
  end
  microtask 1
  value completed 1
  microtask 2
  microtask 3
  normal future running
  normal future completed 1
  microtask 4
it may be that the dart library guys don't want to promise when this thing completes - fear of commitment.
that seems less than ideal, but if it must be, can it be made clearer that it doesn't promise that
the value first becomes available in the next event loop iteration since that seems like the natural way
to interpret the current docs?

Now neither of the docs (Future.value and Future.Future) mention the event or microtask queues. I think the idea is that these are not specified or guaranteed, and should not be relied upon.


missing local over chests variable

https://github.com/Yousei9/Void-Chest-Plus/blob/90864459078e408fde3a77263576a98adc707689/control.lua#L22
Missing local, causes desync (https://forums.factorio.com/79184)

Thanks for reporting.
Quite impressive how long this bug existed without anyone noticing.


CLI Build Fix

Web Platform Detection
When I try to build a Dart CLI with Flutter Stellar SDK, an error occurs, telling me that we can't use package:flutter/foundation.dart because there's no singleton exist in Dart (something like that, I forgot the exact error).
Because we only use the foundation.dart to access kIsWeb, I created two stub files, stub/non-web.dart and stub/web.dart, just to replace the web platform detection.
With the help of dart.library.html, we are able to detect the web platform without importing foundation.dart.
This is just my own workaround, if you have a better way, it would be better.
Dio Overrides
When I try to run the compiled Dart CLI inside Docker container, the call to Soroban server failed due to badCertificateCallback.
So I add the option for Dio overrides just in case someone need it later.

Hello @hasToDev, thank you for adding this PR.


[GA] Generalise fitness type

Feature description
Generalise fitness type so that it is not restricted to f64.
Right now only constraint that fitness is required to fulfill is Ord.
Motivation

It turns out to be quite problematic, as we can't put Ord trait bound on FitnessValueT, because it would eliminate f64 (which does not implement Ord) from being a valid FitnessValueT...


"IndexError: list index out of range" when "Info API down. Falling back to manual fetching..."

Summary
I use more selfbots at once, but when the Info API is detected to be down, "List index out of range" pops out.
Reproduction Steps
By running multiple profiles inside discord.py-self, this issue happens randomly (when the API is detected down).
Code
2024-12-11 16:04:23 - INFO - root - [(usernames...)] Logged in as (username)
2024-12-11 16:04:23 - INFO - root - [(usernames...)] Logged in as (username)
2024-12-11 16:04:23 - INFO - root - [(usernames...)] Logged in as (username)
2024-12-11 16:04:22 - WARNING - discord.utils - Info API down. Falling back to manual fetching...
2024-12-11 16:04:22 - WARNING - discord.utils - Info API down. Falling back to manual fetching...
2024-12-11 16:04:22 - WARNING - discord.utils - Info API down. Falling back to manual fetching...
2024-12-11 16:04:22 - WARNING - discord.utils - Info API down. Falling back to manual fetching...
2024-12-11 16:04:22 - WARNING - discord.utils - Info API down. Falling back to manual fetching...
2024-12-11 16:04:22 - WARNING - discord.utils - Info API down. Falling back to manual fetching...
2024-12-11 16:04:22 - INFO - discord.gateway - Connected to Gateway (Session ID: febd18ff43c8dda074211936d59f6333).
2024-12-11 16:04:22 - INFO - discord.gateway - Connected to Gateway (Session ID: 1eee34af8e4734f966c4ab0ee8849802).
2024-12-11 16:04:22 - INFO - discord.gateway - Connected to Gateway (Session ID: 073e2a618bb80e322463b1c13e2caf55).
2024-12-11 16:04:22 - INFO - discord.gateway - Connected to Gateway (Session ID: 2ed496e79c784eb3da81b51b846aad7c).
2024-12-11 16:04:22 - INFO - discord.gateway - Connected to Gateway (Session ID: 902603b07abe1a1b8d98c315f108c7f5).
2024-12-11 16:04:22 - INFO - discord.gateway - Connected to Gateway (Session ID: 3d8df9872b0ec3334c2dc3ed53cda824).
2024-12-11 16:04:23 - INFO - discord.gateway - Connected to Gateway (Session ID: 78dba3346be3a5f8ef77bd5d17621144).
2024-12-11 16:04:23 - INFO - discord.gateway - Connected to Gateway (Session ID: 237e75b7b993b337db95568102f61a9a).
2024-12-11 16:04:23 - INFO - discord.gateway - Connected to Gateway (Session ID: a6a47c511e646d1020b0c78c928f16a7).
2024-12-11 16:04:23 - INFO - discord.gateway - Connected to Gateway (Session ID: 4bdd4fdbbdb02b678d650c9dd610824f).
2024-12-11 16:04:23 - INFO - discord.gateway - Connected to Gateway (Session ID: 79512ab233ce546790e259c92f0c16aa).
2024-12-11 16:04:23 - INFO - root - [(usernames...)] Logged in as (username)
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Scripts\ActivitySelfbot\ActivitySelfbot.py", line 437, in <module>
    asyncio.run(run_bots(statuses_config))
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\asyncio\runners.py", line 194, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\asyncio\runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\asyncio\base_events.py", line 687, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Scripts\ActivitySelfbot\ActivitySelfbot.py", line 429, in run_bots
    await asyncio.gather(*[bot.start(account['token']) for bot, account in zip(bots, statuses_config['accounts'])])
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\site-packages\discord\client.py", line 857, in start
    await self.login(token)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\site-packages\discord\client.py", line 698, in login
    data = await state.http.static_login(token.strip())
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\site-packages\discord\http.py", line 991, in static_login
    await self.startup()
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\site-packages\discord\http.py", line 562, in startup
    self.super_properties, self.encoded_super_properties = sp, _ = await utils._get_info(session)
                                                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\site-packages\discord\utils.py", line 1446, in _get_info
    bn = await _get_build_number(session)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python312\Lib\site-packages\discord\utils.py", line 1474, in _get_build_number
    build_url = 'https://discord.com/assets/' + re.compile(r'assets/+([a-z0-9]+)\.js').findall(login_page)[-2] + '.js'
                                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^
IndexError: list index out of range

Expected Results
My selfbot should run without issues related to discord.py-self.
Actual Results
IndexError: list index out of range, then my program crashes.
System Information

Python v3.12.4-final
discord.py-self v2.0.0-final
aiohttp v3.9.5
system info: Windows 10 10.0.19045

Checklist

[X] I have confirmed I am using unmodified discord.py-self and not the upstream discord.py.
[X] I have searched the open issues for duplicates.
[X] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
No response

am I the only one doing a simple client login and asyncio gather or is it wrong? I'm confused never ran into your issue


Shared initialized set

This was an attempt to fix #1250 following the proposal there, but this causes other test failures. I am leaving this on a branch/PR for further examination if you think it is salvageable.

First of all, @remkop thanks a lot for the work you put into reviewing this PR, and for doing it so quickly! You have been one of the most responsive maintainers that I have interacted with, and had several great ideas! I'm in a bit of a tight stretch work-wise, I'll try to have a look at your suggestions tomorrow or the day after, but I am interested in implementing the improvements you mention.
Thanks again!

@danielthegray That is great, thanks for helping out with this!
Thinking about some details that may come up during the implementation:

an alternative name to ancestor() may be root(): the meaning is very similar to CommandSpec::root, so using the same name exposes less API surface to users and makes it easier to learn
for the top-level ArgSpec (the one that has scope = INHERIT), there is no ancestor/root, and I suggest that it returns itself rather than null (also similar to how CommandSpec::root behaves)
for ArgSpec objects that are not inherited, the ancestor()/root() method should probably return null

Codecov Report

Merging #1254 (12b26df) into master (755ef48) will decrease coverage by 0.12%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master    #1254      +/-   ##
============================================
- Coverage     93.81%   93.68%   -0.13%
  Complexity      458      458
============================================
  Files             2        2
  Lines          6753     6789      +36
  Branches       1823     1835      +12
============================================
+ Hits           6335     6360      +25
- Misses          126      129       +3
- Partials        292      300       +8

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/picocli/CommandLine.java
93.48% <75.00%> (-0.14%)
317.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 755ef48...12b26df. Read the comment docs.

I saw you pushed a change. Do you want me to merge this, and do any further changes in a separate PR?

No, I essentially rebased the previous work off master (now that the annoying line endings problem was gone, making it easier to switch branches without being left in an un-workable state).
I think it would be best to incorporate your solution as I think it is the most robust way to inherit parameters without needing to worry about possibly not matching the search criteria, and really fixing once and for all this default value overwriting.
I have implemented your suggestion, with a small change which I will detail in the relevant review thread, to allow for infinitely nested subcommands. Let me know what your thoughts are!

If you make that change the PR can be merged. It would be good if we can have additional tests for named options similar to SubcommandTest::testInheritedParameter.

I missed this during my last check, I will add these as well.

Merged with some minor changes.
Thank you very much for the contribution!

@remkop Thank you very much! That was a great catch, that the .root(original.root()) is not needed. I have removed a javadoc comment that I added to that effect, which would be good to merge in as well, I think.

Ok, can you paste a diff/patch of your change here? Since this PR has already been merged I don't think it is possible to merge any subsequent changes.

Sure!
diff --git a/src/main/java/picocli/CommandLine.java b/src/main/java/picocli/CommandLine.java
index 02dd44c2..ca21e508 100644
--- a/src/main/java/picocli/CommandLine.java
+++ b/src/main/java/picocli/CommandLine.java
@@ -9060,7 +9060,6 @@ public class CommandLine {
                 public T hidden(boolean hidden)              { this.hidden = hidden; return self(); }

                 /** Sets whether this option is inherited from a parent command, and returns this builder.
-                 * <b>Do not forget to also set {@link #root()} when setting this, to ensure that it always can track back to the root element.</b>
                  * @since 4.3.0 */
                 public T inherited(boolean inherited)        { this.inherited = inherited; return self(); }

Merged. Thank you for the attention to details!

Thank you again for getting this merged so quickly!


Wiki redaction

Project is quite complex cause it's not just about code. There is hardware and thirdparties.
Wiki must be completed for the existing (few) features.
Then, complete it over time.

Ongoing. Several things have already be written down but still a lot of work there.


Add error messages in require() in TokenNetwork

And turn on the optimizer of the Solidity compiler.  Otherwise, the runtime bytecode becomes too big for deployment.
This is a part of https://github.com/raiden-network/raiden-contracts/issues/81

What about the other solidity files?

There are places missing messages, checked with

I'll see.  By the way assert()s don't fail.

What about the other solidity files?

Will be kept track in #81.

I'll try to keep the optimizer off, by splitting away the messages into a different contract and saving the codesize.

@hackaugusto assert() is implemented using INVALID opcode, which cannot return a message anyway.

When I move the strings to a library, the runtime code becomes bigger.  So a serious split up is necessary https://github.com/raiden-network/raiden-contracts/issues/401 .


Pin d3-selection dep to 3.0.0

zoom.transform uses selection.interupt which is only available in d3-selection 3.0.0 onwards

That doesn’t seem right. First, selection.interrupt is provided by d3-transition, not d3-selection. And second, selection.interrupt has existed since version 0.0.5 of d3-transition; see https://github.com/d3/d3-transition/commit/46bbcd660aec3ee8c8811d57552f8f7711880dab. You probably have a different issue which is duplicate versions of d3-selection installed, preventing d3-transition from modifying d3.selection.prototype.

Ah my bad, I took this comment thread at face value.
fwiw, this issue was that with this structure: (taken from this thread)

Upgrading d3-selection does work, ideally we would have a build conflict somewhere ☹


missing ; (typo)

corrected a typo

These changes look good to me. @davidhassell I think you're the lucky one to merge this in 3w if there are no objections; that would be the 13th, right in time for the workshop. Does that work for you?

And thank you @edzer , that's clearly a typo and the correction is appreciated! :)

Hi @erget - I'm happy to merge this right now - as in 3 weeks we'll already have released CF-1.10, but I'm willing to go out on a limb and say that there is no possible objection to this defect :).

@davidhassell I agree, I've merged it now. If the CF police comes they'll hang us together ;)


Ast override makes is possible to bypass

Parser should be in the forbidden list, because it is still possible to bypass the protection by monkey-patching it:
class ::Parser::AST::Processor
	def on_const(node)
		node = [:String]
	end
end
conn = PG::Connection.open(:dbname => 'rails_edge_development')
conn.exec "delete from console1984_sessions"

Oh totally, good catch @gregmolnar, I'll fix this 🙏

Released 0.1.14 fixing this. Thanks again @gregmolnar.


Use rocksdb instead of leveldb

This switches the key/value database used by hubs from plyvel (leveldb) to lbry-rocksdb, which is an up-to-date binding for rocksdb.

Coverage increased (+0.2%) to 68.874% when pulling 3f1258ee5d587596200e26a2edc39156258194a9 on new-rocksdb into 1e00fb369d31fac980fc239b17a2834cb0631a50 on master.

@jackrobison who should review this? plz assign reviewers

@jackrobison let me know when this is ready for review

@jackrobison Hello there, great job on https://github.com/lbryio/lbry-rocksdb! We're thinking of using that so I'm testing it now. I have a question: Can secondary instances read statistics of the primary? For example, getting the rocksdb.stats property https://github.com/facebook/rocksdb/blob/v6.27.3/include/rocksdb/db.h#L770-L772

@ofek Secondary dbs can read many of the properties, but I'm not sure about rocksdb.stats. Here's a test that checks some of the other properties: https://github.com/lbryio/lbry-rocksdb/blob/master/tests/test_db.py#L720

I found out it was impossible https://github.com/facebook/rocksdb/issues/908#issuecomment-1007048041


feat(portal): 动态获取应用列表

之前应用的列表都是portal-web启动的时候直接从配置文件中读取并保存为类似全局变量的东西，这使portal-web一旦启动就无法重新加载应用列表。这个PR增加了listAvailableApps的API和RPC，并使得portal-web在每次加载网页的时候从这个API中读取可以使用的应用列表（而不是在portal-web启动的时候从配置文件中读取），这使得后台配置文件的修改能够实时生效。

Codecov Report
Base: 64.60% // Head: 64.56% // Decreases project coverage by -0.04% :warning:

Coverage data is based on head (013ee6f) compared to base (417e805).
Patch coverage: 0.00% of modified lines in pull request are covered.

:exclamation: Current head 013ee6f differs from pull request most recent head fc69876. Consider uploading reports for the commit fc69876 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   64.60%   64.56%   -0.05%
==========================================
  Files          99       99
  Lines        2800     2802       +2
  Branches      347      347
==========================================
  Hits         1809     1809
- Misses        946      948       +2
  Partials       45       45

Impacted Files
Coverage Δ

apps/portal-server/src/services/app.ts
10.81% <0.00%> (-0.31%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


agent.getContext() and agent.context.get() both returning null

We are using version ^0.6.1 of dialogflow-fulfillment-nodejs library in our Actions on Google app. During the development and internal testing, everything is working fine. However when we submitted the app for Google Review, the support team is experiencing issues and the bot exits the conversation.
The logs show that the context object returned by agent.getContext() or agent.context.get() is null even though we could see that it's present in the request.body with active lifespan.
Below is the code that I have used to retrieve the context:
   `const sessionAttr = agent.getContext('session_attributes');
    agent.add(sessionAttr.parameters.jobDetails[9]);`

The exception in the log:
dialogflowFirebaseFulfillment 2u600gl7bqkl getContext is deprecated, migrate to context.get dialogflowFirebaseFulfillment 2u600gl7bqkl TypeError: Cannot read property 'parameters' of null     at jobTitle (/srv/index.js:313:31)     at WebhookClient.handleRequest (/srv/node_modules/dialogflow-fulfillment/src/dialogflow-fulfillment.js:313:44)     at exports.dialogflowFirebaseFulfillment.functions.https.onRequest (/srv/index.js:619:11)     at cloudFunction (/srv/node_modules/firebase-functions/lib/providers/https.js:57:9)     at /worker/worker.js:726:7     at /worker/worker.js:709:11     at _combinedTickCallback (internal/process/next_tick.js:132:7)     at process._tickDomainCallback (internal/process/next_tick.js:219:9) dialogflowFirebaseFulfillment 2u600gl7bqkl    undefined
Here is the request.body object for the same intent handler call - here we can see that context "session_attributes" is still active with lifespan of 49:
{         "name": "projects/new-recruitmentbot/agent/sessions/ABwppHHoIPs-VMhwPNN1MXgbvxvNt5kqQXiPPZKkAjeYFqa7y6DYDXEJwMAdHbiL_2g1HSlV4NRBfN_h7S9-iY2L/contexts/session_attributes",         "lifespanCount": 49,         "parameters": {            ...,           "jobDetails": [             "JD001",            ...           ],         }       }
Any help in identifying the issue and fixing it will be appreciated.

Have you found a way to resolve the issue? I am facing the exact same problem independent of the environment. It just started being undefined working from on moment to the other

I ended up using the older version of lib where agent.getContext() is working fine.

I ended up using the older version of lib where agent.getContext() is working fine.

hey, I've tried changing versions in the package.json, cant get any to work. Which version did you roll back to?

I ended up using the older version of lib where agent.getContext() is working fine.

hey, I've tried changing versions in the package.json, cant get any to work. Which version did you roll back to?

@acousticintel version ^0.5.0

I ended up using the older version of lib where agent.getContext() is working fine.

hey, I've tried changing versions in the package.json, cant get any to work. Which version did you roll back to?

@acousticintel version ^0.5.0

I'm using version ^0.5.0 and still getting the same issue:

same problem here. either turn off the environment config in beta or get the contexts manually under agent.contexts

Hello, I solved the problem by turning off "Beta features"


no chat option

i can't enter anything into a chat window

never mind : -) apparently i should click "save" button first

Have you saved the chat?

what is the right prompt schema if i want it to answer history questions as a history teacher? :-)

you can find a list of prompts here: https://github.com/f/awesome-chatgpt-prompts/blob/main/prompts.csv

The thing { } is the one I am asking in the chat , correct?

Yes, when there is a {variable} in the prompt, it will be the chat input.
When there are multiple {variable}, thats when you have to use Format Prompt Values to specify the values for other {variable}.
However in your case, I think you don't need the {variable}. Here's the example prompt that I tested:
I want you to act as a mathematical history teacher and provide information about the historical development of mathematical concepts and the contributions of different mathematicians. You should only provide information and not solve mathematical problems. Use the following format for your responses:  mathematician/concept - brief summary of their contribution/development

That’s awesome , thank you so much !On Apr 29, 2023, at 7:31 PM, Henry Heng @.***> wrote:
Yes, when there is a {variable} in the prompt, it will be the chat input.
When there are multiple {variable}, thats when you have to use Format Prompt Values to specify the values for other {variable}.
However in your case, I think you don't need the {variable}. Here's the example prompt that I tested:
I want you to act as a mathematical history teacher and provide information about the historical development of mathematical concepts and the contributions of different mathematicians. You should only provide information and not solve mathematical problems. Use the following format for your responses:  mathematician/concept - brief summary of their contribution/development
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Update sbt to 1.5.7

Updates org.scala-sbt:sbt from 1.5.6 to 1.5.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Will be merged forward.

Superseded by #318.


Bang version for delete_multiple_objects?

I wonder if there is some special reason for not having a bang version of the delete_multiple_objects function?

Hey there, apologies on the hideously long delay in responding. I'm working on a solution that avoids the crazy amount of duplicate functions required to handle the ! functions and the config optional functions. You can read more about it here: https://github.com/CargoSense/ex_aws/issues/137
I'll not be adding additional functions until that's been done.


Publish 'How to get started with the Chimoney API' on our Community blog

[ ] Publish this blog on our hashnode community blog here.

[ ] To publish join the community team here and publish it as community member.

[ ] The blog is currently written in HTML so you'll need to translate and write it in markdown.

@phyleria Please assign this to me. I will complete it very diligently.

Hi @tvermaashutosh
Unfortunately, this issue has already been claimed.

I want to work on this issue!

Would like to work on the issue

Hi @Aady7
Unfortunately this issue has already been taken.

Is this task done ? If not I would like to do it @phyleria

@phyleria assign me this !


VPN Seed (CPU| Memory) Usage panels don't show data

How to categorize this issue?

/area monitoring
/kind bug
What happened:
For a ManagedSeed cluster:

As shown in the above screenshot, the VPN Seed (CPU| Memory) Usage panels don't show data.
One of the queries behind the panels is rate(container_cpu_usage_seconds_total{pod=~"kube-apiserver-.*|prometheus-.*", container="vpn-seed"}[$__rate_interval]).
There is no such container called vpn-seed in the kube-apiserver Pod (kube-apiserver, apiserver-proxy-pod-mutator) or in the prometheus Pod.
What you expected to happen:
The panels to show data for non-HA and HA control planes.
How to reproduce it (as minimally and precisely as possible):
See above.
Anything else we need to know?:
Environment:

Gardener version: v1.62.2
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Others:

/assign

The VPN dashboard has another issue of panel VPN Shoot CPU Usage.
It only shows the request not show the Actual and Limits

Panel VPN Shoot CPU Usage has the query of Actual and Limits

In Panel VPN Shoot CPU Usage, query of Actual doesn't show data because of the Min step is too low.
The prometheus data-source Scrape interval  is 1m
data:
  prometheus.yaml: |

    global:
      evaluation_interval: 1m
      scrape_interval: 1m

$__rate_interval is always at least four times the value of the Scrape interval. The value of $__rate_interval is defined as max($__interval + Scrape interval, 4 * Scrape interval), where Scrape interval is the “Min step” setting (also known as query*interval, a setting per PromQL query) if any is set. Otherwise, Grafana uses the Prometheus data source’s “Scrape interval” setting.
The Min step setting defines the lower bounds on the interval between data points. If Min step set to 10s, it will not cover sample to calculate the rate. Removing the Min step, the dashboard would show data.

For VPN Shoot CPU Usage - Limits, it doesn't have data as vpn-shoot pod doesn't set cpu limits.

For VPN Seed CPU|Memory Usage, the pod name is vpn-seed-server-.* and container name should be vpn-seed-server


menu bar issues

window buttons like close and minimize go behind the menu bar and is impossible to click the buttons without exiting cairo

What is your OS ?
Version of Windows and architecture (32 or 64bit) ?
You can check this via Win+Pause on your keyboard or right click > properties on 'My PC' icon on desktop or file explorer.
Do you use multiple monitors and what is the status (ticked/unticked) of the option "Show menu bar on multiple monitors" in Cairo ?
Please check you have the latest Cairo version. In Cairo menu, use check update. If no update available, compare version number to the latest milestone just to be sure.
Did you try to minimize then restore the window ?
You can do this first in native shell, then launch Cairo.
Or in Cairo you can do win + arrow down once then win + arrow up once.
It will minimize then maximize the window, without the use of the unreachable buttons.
After that, do the window still have wrong position (under Cairo menu bar) ?

OS: Windows 10
Arcitrcture: 64 Bit
Monitor setup: 1 screen
I am using cairo for the sole
pourpose because my native shell isnt working.
I say unreachable buttons because in some windows you cannnot click the
button to the right of the minimize button
also called "grouping"
-Fadhlan
On Mon, Mar 12, 2018, 12:56 AM mdmk<EMAIL_ADDRESS>wrote:

What is your OS ?
Version of Windows and architecture (32 or 64bit) ?
You can check this via Win+Pause on your keyboard or right click >
properties on 'My PC' icon on desktop or file explorer.
Do you use multiple monitors and what is the status (ticked/unticked) of
the option "Show menu bar on multiple monitors" in Cairo ?
Please check you have the latest Cairo version. In Cairo menu, use check
update. If no update available, compare version number to the latest
milestone just to be sure.
Did you try to minimize then restore the window ?
You can do this first in native shell, then launch Cairo.
Or in Cairo you can do win + arrow down once then win + arrow up once.
It will minimize then maximize the window, without the use of the
unreachable buttons.
After that, do the window still have wrong position (under Cairo menu bar)
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372134848,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVAcpVQKE7G__iuNGl0GnBKcTXNzeks5tdWU3gaJpZM4SkHeG
.

I suggest you to repair your native shell first.
Unless @dremin want to investigate this wrong behavior further.
An easy way to repair your shell is to reinstall OS.
It is possible to do this without losing documents and third party applications.
To do so, follow this procedure carefully.

ok thanks for your advice
-Fadhlan
On Mon, Mar 12, 2018, 7:05 PM mdmk<EMAIL_ADDRESS>wrote:

I suggest you to repair your native shell first.
Unless @dremin https://github.com/dremin want to investigate this wrong
behavior further.
An easy way to repair your shell is to reinstall OS.
It is possible to do this without losing documents and third party
applications.
To do so, follow this procedure
http://www.intowindows.com/repair-windows-10-install-without-losing-apps-data/
carefully.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372287425,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVEQSg7_9lK8W4FoUKXEhIib-gMtoks5tdmR6gaJpZM4SkHeG
.

There are other ways to repair shell but not as straightforward.
After doing this, test Cairo again.
If bug is gone, problem come from wrong Windows installation.
If not, we'll take a look further.
Let us know how things turn out.

ok
-Fadhlan
On Mon, Mar 12, 2018, 7:08 PM mdmk<EMAIL_ADDRESS>wrote:

There are other ways to repair shell but not as straightforward.
After doing this, test Cairo again.
If bug is gone, problem come from wrong Windows installation.
If not, we'll take a look further.
Let us know how things turn out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372288211,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVBGVOJaMjyObXjVHLvXcHChouahXks5tdmU-gaJpZM4SkHeG
.

my native shell i looked up the problem is that shcore.dll is somehow not
being loaded but i checked using dism that the file is ok
-Fadhlan
On Mon, Mar 12, 2018, 7:09 PM muhammad fadhlan<EMAIL_ADDRESS>wrote:

ok
-Fadhlan
On Mon, Mar 12, 2018, 7:08 PM mdmk<EMAIL_ADDRESS>wrote:

There are other ways to repair shell but not as straightforward.
After doing this, test Cairo again.
If bug is gone, problem come from wrong Windows installation.
If not, we'll take a look further.
Let us know how things turn out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372288211,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVBGVOJaMjyObXjVHLvXcHChouahXks5tdmU-gaJpZM4SkHeG
.

Did you try all these steps ?
You can also do them in failsafe mode if it does not work.

i currently dont have access to my computer atm because exams
-Fadhlan
On Mon, Mar 12, 2018, 7:14 PM mdmk<EMAIL_ADDRESS>wrote:

Did you try all these steps
https://answers.microsoft.com/en-us/windows/forum/windows_10-files/the-shellhistory-folder-crashes-explorerexe/f652ac3c-16ce-4590-8479-459d7faf3985
?
You can also do them in failsafe mode if it does not work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372289682,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVA7oEOSswQdcqpJ0C3KufNHv2_cpks5tdmaugaJpZM4SkHeG
.

No problem.
Come back when you have time.

ill finish my exams on 16 Mar 2018
-Fadhlan
On Mon, Mar 12, 2018, 7:15 PM muhammad fadhlan<EMAIL_ADDRESS>wrote:

i currently dont have access to my computer atm because exams
-Fadhlan
On Mon, Mar 12, 2018, 7:14 PM mdmk<EMAIL_ADDRESS>wrote:

Did you try all these steps
https://answers.microsoft.com/en-us/windows/forum/windows_10-files/the-shellhistory-folder-crashes-explorerexe/f652ac3c-16ce-4590-8479-459d7faf3985
?
You can also do them in failsafe mode if it does not work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372289682,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVA7oEOSswQdcqpJ0C3KufNHv2_cpks5tdmaugaJpZM4SkHeG
.

ok, thanks!
-Fadhlan
On Mon, Mar 12, 2018, 7:16 PM mdmk<EMAIL_ADDRESS>wrote:

No problem.
Come back when you have time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372290172,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVDHvD0vXetdXm0w8FnYiGcFkhqwEks5tdmctgaJpZM4SkHeG
.

You're welcome.

When installing Cairo did you select the option to replace Explorer?

hmm i wil check later
-Fadhlan
On Tue, Mar 13, 2018, 6:21 AM Sam Johnson<EMAIL_ADDRESS>wrote:

When installing Cairo did you select the option to replace Explorer?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-372495007,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVKM99YVzIFi75cE2WENKOS4OArIyks5tdwMAgaJpZM4SkHeG
.

ok so i seem to have found a solution which is to use a little program called always-on-top.exe
to make windows stay on top of other windows using Ctrl+Space

When the Windows shell is running (which it appears to be in your screenshot based on the presence of the Action Center button), Cairo sends it commands to reserve space on screen. I suspect since your Windows shell is broken, this isn't working correctly. If you reinstall Cairo with the option to replace Explorer, then log off/log on, I suspect you should see better results. If you're able to try this at some point, I'd be interested what happens.

ok im trying that now
-Fadhlan
On Sun, Mar 18, 2018, 8:57 AM Sam Johnson<EMAIL_ADDRESS>wrote:

When the Windows shell is running (which it appears to be in your
screenshot based on the presence of the Action Center button), Cairo sends
it commands to reserve space on screen. I suspect since your Windows shell
is broken, this isn't working correctly. If you reinstall Cairo with the
option to replace Explorer, then log off/log on, I suspect you should see
better results. If you're able to try this at some point, I'd be interested
what happens.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cairoshell/cairoshell/issues/98#issuecomment-373966870,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkmVIMUEmzbS1__hTrI7X3xt10uvFPJks5tfb8KgaJpZM4SkHeG
.

but my explorer shell is still broken so i cant open coltrol panel

but now window controls dont go behind the menu bar, which is great

Glad to hear it! I'll close this out.


Array of NSStrings

Any ideas how to parse this:
{   "success": [     "0124555",     "0124555"   ],   "failure": [     "124555"   ] }
My model seems to cause an error: Terminating app due to uncaught exception 'Type not allowed', reason: 'NSArray type not supported for SyncLocationsResponse.success'
@interface SyncLocationsResponse : JSONModel
@property (strong, nonatomic, nullable) NSArray *success;

@property (strong, nonatomic, nullable) NSArray *failure;

@end
My suspicion is it can be solved using code in the implementation file of the JSONModel

My mistake. It does it automatically.


Exposure of SUPABASE_SERVICE_ROLE_KEY on GitHub

Description: A team member accidentally published the SUPABASE_SERVICE_ROLE_KEY on GitHub, which is a secret key used by the Supabase service to authenticate requests and perform operations on behalf of a service role. This key is used to grant permissions to specific resources within the Supabase project. As a result, the key is now accessible to anyone who has access to the repository, which can lead to potential security breaches and unauthorized access to team resources.
Actions Taken:

The team member immediately removed the key from the repository to prevent unauthorized access.
The team member revoked the exposed key from the Supabase service to prevent anyone from using it to access team resources.
The team member generated a new key and updated the service to use the new key.
The team reviewed the code to ensure that no other sensitive information is being exposed.
The team member notified the team about the issue and the actions taken to fix it.

It is important for the team to be aware of the potential risks associated with accidentally publishing sensitive information on public repositories. It is recommended to implement security measures such as:

Use environment variables to store sensitive information instead of hardcoding them in the code.
Implement code reviews and automated tools to detect and prevent the publishing of sensitive information.
Provide training and education to team members on the importance of keeping sensitive information secure.

By taking these steps, the team can help prevent potential security breaches and protect sensitive information from unauthorized access.

Hi @arpagon, sorry to hear that. We use .env files for local development.
Would you like to make a PR to our README to instruct users and suggest the best way to go about this?

Let's clarify something because I might be wrong.
https://github.com/Helicone/helicone/blob/c9b72c8df9c42335bc5756afa3d48ec5c17cc2af/worker/wrangler.toml#L6
Isn't the production SUPABASE_SERVICE_ROLE_KEY?

Now I understand that this is not the production key. ok, I will collaborate a little in the documentation. Thank you very much.

@arpagon luckily this specific key is only for localhost, so leaking that speicifc key is OK

Hopefully this will prevent this from happening again
https://github.com/Helicone/helicone/pull/136


[WFLY-18938] Add a profile to disable wildscribe generation

Issue: https://issues.redhat.com/browse/WFLY-18938
Add a profile to disable wildscribe generation

Thanks @soul2zimate, but I'm going to close this as @rhusar is correct. Sorry to have wasted your time! Please feel free to reopen if you disagree.


move definitions out of routes.json

part of #635. Closes #358

@lolasc thanks for the thumbs up :) If you could use GitHub's review feature, that'd be great. It's a good way for us to see that you are looking at our PRs helping spotting oversights and that you are interested in getting involved ;)


Resume and reprod of reproducibility for Hunayuan

I notice that hunyuan finetuning script seems not support resume_from_checkpoint mehod and it is signed as "TODO". Do you plan to add this. Besides, I wonder how to use your inference API to generate the same results in finetuning validation by the checkpoint of finetuning? Thank you!

Support for this has been added in #139. Could you give it a try? Marking the issue as resolved, but if it does not work as expected, please do re-open and ping @sayakpaul