SecurityException: Failed to read sufficient bytes from /dev/urandom

In our app we are using OkHttp WebSockets from ReactNative. And sometimes okhttp.ws. WebSocketCall.create method crash with "SecurityException: Failed to read sufficient bytes from /dev/urandom. Expected: 1024, actual: 0"
The crash happens only on Android 5.x.x. After some source codes research I found out that in Android 5.x.x other Security Provider - OpenSSL than is used in Android 4.x and Android 6+
OpenSSL uses RAND_load_file function (https://github.com/openssl/openssl/blob/6974fca49d9d0b110c02c83a7bbe01907472ac5e/crypto/rand/randfile.c) to read entropy stored in file /dev/urandom in order to initialize pseudo-random number generator with a truly random seed. Apparently in some cases either file can't be opened for read or only 0 bytes are read.
Unfortunately I can't understand the root cause why only 0 bytes are read or file fails to be opened.
However, I think that in WebSocketCall you should use not a default SecureRandom constructor, but rather instantiate SecureRandom with a more reliable Security Provider like those used in Android 4 or Android 6 (need to derive from SecureRandom and override protected constructor).
The problem reproduces on a range of Android 5.x.x devices. Here's top:
8%  Redmi Note 3
8%  SM-N9005
6%  Lenovo A6020a40
5%  LG-H955
5%  m2
69% 40 other machines
Here's the complete exception stacktrace:
com.android.org.conscrypt.OpenSSLRandom.seedOpenSSLPRNGFromLinuxRNG in OpenSSLRandom.java::75
com.android.org.conscrypt.OpenSSLRandom.selfSeedIfNotSeeded in OpenSSLRandom.java::61
com.android.org.conscrypt.OpenSSLRandom.engineNextBytes in OpenSSLRandom.java::37
java.security.SecureRandom.nextBytes in SecureRandom.java::271
okhttp3.ws.WebSocketCall. in WebSocketCall.java::64
okhttp3.ws.WebSocketCall. in WebSocketCall.java::54
okhttp3.ws.WebSocketCall.create in WebSocketCall.java::46
com.facebook.react.modules.websocket.WebSocketModule.connect in WebSocketModule.java::115
java.lang.reflect.Method.invoke in Method.java::-2
java.lang.reflect.Method.invoke in Method.java::372
com.facebook.react.bridge.BaseJavaModule$JavaMethod.invoke in BaseJavaModule.java::319
com.facebook.react.cxxbridge.JavaModuleWrapper.invoke in JavaModuleWrapper.java::139
com.facebook.react.bridge.queue.NativeRunnable.run in NativeRunnable.java::-2
android.os.Handler.handleCallback in Handler.java::815
android.os.Handler.dispatchMessage in Handler.java::104
com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage in MessageQueueThreadHandler.java::31
android.os.Looper.loop in Looper.java::194
com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run in MessageQueueThreadImpl.java::196
java.lang.Thread.run in Thread.java::831

That’s odd. As specified /dev/urandom is always supposed to provide entropy.


Visualization code wanted

Hi, Thanks for your great work!
In your paper, attention maps of particular query points are shown. Could you share this visualization code?
More specifically, did you implement an interactive interface to do the visualization?
Many thanks again.

Please find the visualization code here.


Source Root Directory in Output Path when using tsconfig.json

Running into a rather interesting issue when working with the Aurelia team to resolve some issues I was having with the TypeScript build.  It appears as though when building using tsconfig.json the root directory is being appended to the output path.  For example, when building the typescript in 'src' and outputting the javascript to 'dist', the compiled javascript will be sent to 'dist/src'.  [Here is an example gulpfile] (https://github.com/cmichaelgraham/skeleton-navigation-typescript-vscode/blob/master/build/tasks/build.js) from the Aurelia team which suffers from the same issue.

The directory that contains the tsconfig file is automatically set as the base directory. You can move that file to the src directory or set the sourceRoot property in the compilerOptions section of the tsconfig.json file to 'src'. Does one of these options solve your issue?

I really would rather not move the tsconfig.json as I believe its rightful place is in the root directory of the projects (much like config.json and packages.json).
I tried adding the sourceRoot to the compiler options of tsconfig.json and was met with the message and the issue persisting:

sourceRoot isn't supported any more. Use sourceRoot option of gulp-sourcemaps instead.

Taking that suggestion into consideration, I updated my call to gulp-sourcemaps to include the sourceRoot option, but to no avail

Sorry, I meant the rootDir option instead of sourceRoot.

I really would rather not move the tsconfig.json as I believe its rightful place is in the root directory of the projects (much like config.json and packages.json).

And what if you wanted to build the e2e tests into test\e2e\dist, like the skeleton project does with Babel?
To me it seems more appropriate to put the tsconfig file in the src folder. The reason why I personally don't do it this way yet is because I don't want to rely on the atom-typescript-specific filesGlob functionality, so I'm currently using exclude with tsconfig in the project root. If I move tsconfig to src, then I'd have to manually list files or use filesGlob until this enhancement is implemented.
TBH, I'm in two minds about this.

@ivogabe that appears to have done the trick!

I'm having a bit of trouble with this also.
My file structure is like so:
- project
  - ts
  - js

So, I call the tsc command line from js:
tsc -m commonjs --outDir ./ -p ../ts --rootDir ../ts

It continues to copy the parent ts directory.
So, I tried this in the ts directory and it seems to work now:
tsc -m commonjs --outDir ../js --rootDir ts

Not sure if that is a bug or not.

Also, for some reason it rebuilds the who project. Seems to defeat the purpose of watch :-(

@glen-84 I usually put tests in the lib directory, something like this:
lib/
  client/
    ..
  test/
    ..
  tsconfig.json

I guess your lib is my src. I suppose it's something to consider. Thanks.

I guess your lib is my src.

Yes, that's right. Let me know wether this works for you


add 'from' keyword

Hi @Gheoan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@Gheoan, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

@zhengbli rebased and rady to merge

@Gheoan Thanks!


Refactor IR.scala to mitigate problems with calling nested classes from Java

TreeToIr translates from new AST to Enso's internal IR. The class has been written in Java, by design and on purpose (reasons omitted), but it had to make use of the existing class hierarchy defined in IR.
While Scala and Java can co-exist, calling the former from the latter is not pleasant when dealing with nested classes. And Scala code often makes use nested classes. This is easily demonstrable just by looking at the imports of TreeToIr:

i.e. we can't use the typical "dot" syntax for nested classes. This makes it apparently impossible to recognize for IDE's, as demonstrated in the screenshot.
There are possible solutions (most of them hard to enforce because out of our control):

Make sure Scala emits better InnerClasses attribute for deeply (2+ level) nested inner classes. Not sure if this is even possible given the rather complex interop logic (e.g. https://discord.com/channels/@me/959434339182067712/1116738522003550269). Will file a ticket in the Scala compiler.
Make javac understand that nested classes belonging to companion objects are called Foo$Bar$Baz and not Foo$Bar$$Baz. Also unlikely to happen because most of the implementation follow blindly the $ pattern.
Fix IDE - unlikely to happen in a timely manner, given that IntelliJ is closed source
Refactor IR to multiple classes.

Due to the lack of other alternatives, this ticket proposes to pursue option 4. This bug slows down developer productivity quite significantly.

Same situation with Runtime.scala


Synchronize incorrect layout

Something I noticed while testing this locally was that this line seems to synchronize against the current layout instead of the incoming layout. I would think that you would want to synchronize the next children with their layout and not the current layout. Changing it to nextProps.layout seems to fix a lot of issues I had with children not inheriting the correct layout from the grid component parent.
Is this intentional or could this possibly be a typo during the recent big update?

This appears to be a mistake.
Could you explain how you are using it? This appears to be also related to the open-ended question in #156. For example, I could see the following flow:

Developer passes an initial layout into RGL
User drags items around. this.state.layout now !== this.props.layout.
User triggers an action that creates a new layout item.
If the grid were synchronized using this.props.layout as a base, all the changes the user made to the grid before adding the new item would be lost.

Perhaps we should be checking this._pendingState (or storing this ourselves because I don't think that's a stable API)?

We have it setup so any change that is made to RGL triggers an action that creates a new layout that is passed back into RGL. So for the second step you described, once an item is dropped, we dispatch an event that takes the new layout, does stuff e.g. XHR, then pass it back to RGL as the next layout prop so this.state.layout should equal this.props.layout.
The reason I looked into this originally was we have multiple dashboard that can be swapped out (they all use RGL) and upon switching to a dashboard making the next dashboard layout incorrectly. Upon looking at React DevTools, the layout prop from ReactGridLayout didn't match up with the _grid props of it's children.
:disappointed:  I know that may not have clarified too much ... but I'm still looking into the issue a bit myself, but the change in the PR did fix all the issues I was noticing with layouts being out of sync.

Understood. So it seems we should be checking if both children & props have changed in that tick, and if they have, use the new layout.
Could you try:
componentWillReceiveProps(nextProps: Object) {
  let newLayout;
  // Allow parent to set layout directly.
  if (!isEqual(nextProps.layout, this.props.layout)) {
    newLayout = synchronizeLayoutWithChildren(nextProps.layout, nextProps.children,
                                                  nextProps.cols, nextProps.verticalCompact);
  }
  // If children change, regenerate the layout.
  else if (!nextProps.children.length !== this.props.children.length) {
    newLayout = synchronizeLayoutWithChildren(this.state.layout, nextProps.children,
                                              nextProps.cols, nextProps.verticalCompact);
  }
  if (newLayout) {
    this.setState({layout: newLayout});
    // Call back so we can store the layout
    this.props.onLayoutChange(newLayout);
  }
}

And let me know if it fixes your bug?

:+1: :beers: Yup that worked!

0.10.6

:100: Thanks again!


Can't install on mac os arm Mac OS Sonoma 14.4

gem install debase -v 0.2.5.beta2

Macbook Pro M1 Pro
Mac OS 14.4
Ruby 3.3.0 (same error happens on Ruby 2.7.8)

compiling debase_internals.c
debase_internals.c:319:25: warning: initializing 'rb_control_frame_t *' (aka 'struct rb_control_frame_struct *') with an expression of type 'const rb_control_frame_t *' (aka 'const struct rb_control_frame_struct
*') discards qualifiers [-Wincompatible-pointer-types-discards-qualifiers]
    rb_control_frame_t *start_cfp = RUBY_VM_END_CONTROL_FRAME(TH_INFO(thread));
                        ^           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
debase_internals.c:770:3: error: incompatible function pointer types passing 'void (VALUE, VALUE)' (aka 'void (unsigned long, unsigned long)') to parameter of type 'VALUE (*)(VALUE, VALUE)' (aka 'unsigned long
(*)(unsigned long, unsigned long)') [-Wincompatible-function-pointer-types]
  rb_define_module_function(mDebase, "set_trace_flag_to_iseq", Debase_set_trace_flag_to_iseq, 1);
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/douglas/.asdf/installs/ruby/3.3.0/include/ruby-3.3.0/ruby/internal/anyargs.h:338:142: note: expanded from macro 'rb_define_module_function'
#define rb_define_module_function(mod, mid, func, arity)    RBIMPL_ANYARGS_DISPATCH_rb_define_module_function((arity), (func))((mod), (mid), (func), (arity))
                                                                                ^~~~~~
/Users/douglas/.asdf/installs/ruby/3.3.0/include/ruby-3.3.0/ruby/internal/anyargs.h:274:1: note: passing argument to parameter here
RBIMPL_ANYARGS_DECL(rb_define_module_function, VALUE, const char *)
^
/Users/douglas/.asdf/installs/ruby/3.3.0/include/ruby-3.3.0/ruby/internal/anyargs.h:256:72: note: expanded from macro 'RBIMPL_ANYARGS_DECL'
RBIMPL_ANYARGS_ATTRSET(sym) static void sym ## _01(__VA_ARGS__, VALUE(*)(VALUE, VALUE), int); \
                                                                       ^
debase_internals.c:773:3: error: incompatible function pointer types passing 'void (VALUE, VALUE)' (aka 'void (unsigned long, unsigned long)') to parameter of type 'VALUE (*)(VALUE, VALUE)' (aka 'unsigned long
(*)(unsigned long, unsigned long)') [-Wincompatible-function-pointer-types]
  rb_define_module_function(mDebase, "unset_iseq_flags", Debase_unset_trace_flags, 1);
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/douglas/.asdf/installs/ruby/3.3.0/include/ruby-3.3.0/ruby/internal/anyargs.h:338:142: note: expanded from macro 'rb_define_module_function'
#define rb_define_module_function(mod, mid, func, arity)    RBIMPL_ANYARGS_DISPATCH_rb_define_module_function((arity), (func))((mod), (mid), (func), (arity))
                                                                                ^~~~~~
/Users/douglas/.asdf/installs/ruby/3.3.0/include/ruby-3.3.0/ruby/internal/anyargs.h:274:1: note: passing argument to parameter here
RBIMPL_ANYARGS_DECL(rb_define_module_function, VALUE, const char *)
^
/Users/douglas/.asdf/installs/ruby/3.3.0/include/ruby-3.3.0/ruby/internal/anyargs.h:256:72: note: expanded from macro 'RBIMPL_ANYARGS_DECL'
RBIMPL_ANYARGS_ATTRSET(sym) static void sym ## _01(__VA_ARGS__, VALUE(*)(VALUE, VALUE), int); \
                                                                       ^
1 warning and 2 errors generated.
make: *** [debase_internals.o] Error 1

make failed, exit code 2

I had some success by reducing that error to a warning with:
bundle config build.debase --global "--with-cflags=-Wno-error=incompatible-function-pointer-types"

May have needed a gem pristine debase --version 0.2.5.beta2 afterwards, hard to tell.

Try this
gem install debase -v 0.2.5.beta2 -- --with-cflags=-Wno-incompatible-function-pointer-types

I had some success by reducing that error to a warning with:
bundle config build.debase --global "--with-cflags=-Wno-error=incompatible-function-pointer-types"

May have needed a gem pristine debase --version 0.2.5.beta2 afterwards, hard to tell.

Yes this helped me too

I had some luck using Xcode command line tools version 15.1 rather than version 15.3

My issue was bundler version 2.5.7. Rolling back to 2.5.6 and using 0.2.5.beta2  worked for me.

My issue on bundle install was bundler version 2.5.7. Rolling back to 2.5.6 and using 0.2.5.beta2  worked for me.

macOS: Sonoma 14.4.1
Ruby: 3.3.0
This workaround don't worked for me, I got the same errors again. I tried all versions from 2.5.7 to 2.5.9, all the same result.

https://github.com/ruby-debug/debase/pull/110

For those using bundler add this to your .bundle/config:
BUNDLE_BUILD__DEBASE: "-- --with-cflags=-Wno-incompatible-function-pointer-types"

I've also discovered that this issue is also present for rails-debug-ide and grpc (holy).
BUNDLE_BUILD__RUBY_DEBUG_IDE: "-- --with-cflags=-Wno-incompatible-function-pointer-types"
BUNDLE_BUILD__GRPC: "-- --with-cflags=-Wno-incompatible-function-pointer-types"

Works for me.

For those using bundler add this to your .bundle/config:
BUNDLE_BUILD__DEBASE: "-- --with-cflags=-Wno-incompatible-function-pointer-types"

This didn't work for me, unfortunately; I was very hopeful the extra -- would somehow be it! I still get "incompatible function pointer types …"-type errors in the resulting gem_make.out:
debase_internals.c:770:3: error: incompatible function pointer types passing 'void (VALUE, VALUE)' (aka 'void (unsigned long, unsigned long)') to parameter of type 'VALUE (*)(VALUE, VALUE)' (aka 'unsigned long (*)(unsigned long, unsigned long)') [-Wincompatible-function-pointer-types]
  770 |   rb_define_module_function(mDebase, "set_trace_flag_to_iseq", Debase_set_trace_flag_to_iseq, 1);
      |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

It seems debase really doesn't want to pass the Bundler config down the line, as opposed to gem install debase -- … (which works fine). Weird.

Thanks, @hurricup :heart:


Resolve pytorch warnings of nonzeros

Describe the feature
Resolve pytorch warnings of nonzeros
Motivation
In pytorch 1.6, the code reports many warnings of deprecated usages of nonzeros during training.

Fixed by #330.


added footer bar [skip ci]

Overviews

spaces removed
Release Notes fixed
-added the border left to the Follow QGIS secdtion

Lgtm


Revert "Move to uv as package manager"

Reverts david26694/cluster-experiments#195

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.46%. Comparing base (3b3463e) to head (d6ea955).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##           main     #204       +/-   ##
=========================================
+ Coverage      0   96.46%   +96.46%
=========================================
  Files         0       16       +16
  Lines         0     1529     +1529
=========================================
+ Hits          0     1475     +1475
- Misses        0       54       +54

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


#936: Fix lost whitespaces (including linebreaks) on unmarshalling

Fix lost whitespaces (including LF) after unmarshalling of @XmlAnyElement + @XmlMixed content, which was caused by the mixed flag being lost in the middle of the processing, thus calling processText(ignore:true) and rejecting significant whitespaces, including linebreaks.
Fixed by keeping the prev(ious) mixed flag state during State::pop(). This was the least impacting place I could find and at that point, current will point to previous anyways. Please review for other possible scenarios and side-effects (that I tried to cover with automated tests).
I believe this can fix #936 , #1246 and #909 .

addressed in #1630


Unexpected Limit behavior

Subject of the issue
Reported in Discord
Currently the following produces somewhat surprising results
https://testnet-idx.algonode.cloud/v2/applications/83648221/logs?limit=500
https://testnet-idx.algonode.cloud/v2/applications/83648221/logs?limit=1
Where limit 1 returns no results.
I believe this is because the transaction search applies the limit and the filter here:
https://github.com/algorand/indexer/blob/develop/api/handlers.go#L542-L550
further limits the result set, so the limit is how many rows to retrieve in the db rather than the actual limit of log values to get back.

Thank you for submitting this for me, @barnjamin.


Fixed a few lint/format/convention issues

This builds on #2 but fixes up some lint/format issues I encountered.

dart fmt all the code. This makes it easier for the next person to contribute, without worrying about changing whitespace everywhere (which is exactly what my IDE did when I tried to make a simple change :))
Remove the {android,ios,linux,macos,windows} directories in the root. They aren't used as this is a pure dart library. The ones in the example directory are still there.
Few other lint issues.

I checked the example (and from my own project), and everything still works great.

FYI There are other things I didn't do. For example, some libraries can be bumped, such as storybook, but would require minor code changes.

I rely on IntelliJ's formatting but it's inconsistent sometimes. Thank you, looks great.

I use vscode, but there is a standard dart formatting style, that I hope IntelliJ is/can use.
but cool thanks.

Oh it seems I manually formatted this one, my bad. Btw I published 0.2.0 that should have these PRs. Thanks again!


Enhancement: If your having problems with the shaders

if your having problems with the shaders on mac or linux and such. Then use this flxshader-fix this will fix the shaders and i put a tutorial on the bottom to see how to add it on there. @nebulazorua i am not trying to self promote in your repository i am only trying to help people if the shaders are crashing. @nebulazorua if you want to you can take this down i am just trying to help people

and if your also having fps issues with it being a bit laggy just do lime build [your platform] -debug and it will cause less lag issue i hope you guys have a good day.

this.. doesnt fix anything tho?
All it does is let you use them at compile-time iirc, and I'll be adding @gedehari's one at some point

gedeharis's shader? will it work for the shaders that you ported


Better handling of montage files

When reading montage files, users seem to face a number of issues.

Documentation of the different file formats is not clear. For example, *.hpts should not have a header and a column of type. This is not clearly explained anywhere. You cannot figure this out without looking at the code.

Inconsistent handling of numpy versions when reading text files. Here it is handled properly: https://github.com/mne-tools/mne-python/blob/master/mne/channels/montage.py#L325-L328 but this is not the case in all calls. Ideally, this should be abstracted away into a private helper function.

maybe an easy issue for @fmamashli to solve who encountered this in the first place

FYI the read_montage is currently being deprecated in
https://github.com/mne-tools/mne-python/pull/6764
to make sure we are addressing the issue @fmamashli
https://github.com/fmamashli can you clarify the usecase? How was the
.hpts
file produced? what does it contain? points in what coordinate system?

we bumped the minimum version of numpy. We no longer need to keep both versions of skip_row / header.

We digitized the channels using a new system: Localite.
.hpts: we followed MNE manual: EEG, channels name, coordinates (x,y,z) in mm.
The coordinate system is RAS.
The problem was that in MNE-python, there was no good documentation on the format of the files and what exactly it should contain. like headers, columns, coordinate system.

also just to add to what @fmamashli said, see the documentation of mne-c. It is clearly mentioned what the format should look like.

if you use the current master you can try this command:
https://mne.tools/dev/generated/mne.channels.make_dig_montage.html#mne.channels.make_dig_montage
provided you have the locations already read in python
you could ideally specify that the points are in MRI RAS and then it should
work just fine (hopefully).
share some fake data if needed.

We managed for now. But I would say you need a documentation page analogous to this one for reading in raw files.

should be fine now with:
https://mne.tools/dev/auto_tutorials/misc/plot_sensor_locations.html#sphx-glr-auto-tutorials-misc-plot-sensor-locations-py
https://mne.tools/dev/overview/implementation.html#dig-formats

Cool !


Plugins list

I think we're missing one very important thing in documentation - list of all plugins that are available. Now a potential user has to either browse the repos inside organization or check all packages published on hex from our account

We can also put experimental plugins there (marked as experimental), categorize them all, and also make a list of needed plugins in order of importance

closed as of https://github.com/membraneframework/guide/pull/44


Allow Name Filters

I would like to be able to show/hide or change color to notes that match a specified filter.
Examples:

What if I wanted to remove all notes that were named "README"?
What if I wanted to highlight all notes that had "2024-09-10" in the title?
What if I wanted to highlight all pdf files?
What if I wanted to ignore all files within certain directories that match a specific regex?

Thank you for your feedback, and filter on name and color group support might be added in a future release. Currently, we are focused on providing a complete overview of the vault. PDF files may eventually be supported in the same way as Excalidraw files.


Update keyboard leds when pressed

Please let me know if you have a better way of doing this. This was the easiest/minimal changes that I could think of

This looks good. wlc_keyboard_update_leds could take bitmask instead of the wlc_keyboard struct though. You can also drop wlc_ prefix and remove the function prototype from keyboard.h and declare the function simply as static in keyboard.c since that's the only place it's used from.

Thanks for the suggestions. That is much better

Thanks, I'll give it a go later today.

Hmm so I received an email saying add an assert, but now I can't see it on gh. I think I added in the right place...

Yup, that's correct. Thanks


Electron not installing

I'm trying to clone and run electron quick start example and I'm unable to install electron itself
><EMAIL_ADDRESS>postinstall /Users/iMac Casa/electron-quick-start/node_modules/electron-prebuilt
> node install.js

/Users/iMac Casa/electron-quick-start/node_modules/electron-prebuilt/install.js:22
  throw err
  ^

Error: end of central directory record signature not found
    at /Users/iMac Casa/electron-quick-start/node_modules/yauzl/index.js:167:14
    at /Users/iMac Casa/electron-quick-start/node_modules/yauzl/index.js:473:5
    at /Users/iMac Casa/electron-quick-start/node_modules/fd-slicer/index.js:32:7
    at FSReqWrap.wrapper [as oncomplete] (fs.js:614:17)
npm ERR! Darwin 15.3.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v5.5.0
npm ERR! npm  v3.5.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>postinstall: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the electron-prebuilt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs electron-prebuilt
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls electron-prebuilt
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/iMac Casa/electron-quick-start/npm-debug.log

I've already checked my node and npm version and they're both up to date, I'm also attaching here the debug log
npm-debug.log.zip

same here.
already tried #69 with no success.
sh: node: command not found
npm ERR! Darwin 15.2.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "electron-prebuilt" "-g"
npm ERR! node v5.5.0
npm ERR! npm  v3.6.0
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn

@AndreAmo that error sounds like the electron zip download failed and the zip got corrupted. you might try again, it could just be a hiccup with your network or with npm

The comment at #69 allowed me to install Electron but I think it's still an issue, feel free to close it or to leave it open though

I'm not sure whether it was an download error or not but before opening the issue I tried several times in different moments, anyway I just tried the solution in the comment at #69 and the install went through. Thanks for your help

Resolved!


Improve custom form validation response message

Currently when adding custom form validation as described in the docs you can only return false if a certain field fails validation.
if ($data["field1"] != "value1") {
    return false;
}

return true;

This pull request would allow you to return a custom validation error message like so:
if ($data["field1"] != "value1") {
    return "Field1 did not have the correct value, please try again.";
}

return true;

This allows you to display this error message on the client side and give a user more information on why the form failed validation.
I added the conditional return false===$validation ? false : ['error' => $validation, 'data' => $data]; so that this change will not be breaking for anyone still using the current validation response.

Thanks for the contribution, but I don't think this is needed. What you can also do in the validation file:
$this->app->stop(['error' => 'Field1 did not have the correct value!'], 412);

412 is the http error code for 'Precondition Failed'

You're right that is cleaner. I'm still learning the ins and outs of Cockpit!


[OTHER] Update Contributing Guidelines

What would you like to share or ask?
These are some of the points that I will incorporate along with other considerations:

There are no clear guidelines to clarify what a contributor should do when there are multiple languages associated with a resource.

There are no clear guidelines mentioning the database folder as the designated location to add proper link resources.

Additionally, using present tense in commit messages is considered a basic convention (although this is optional, as there are no strict rules), as the contributing guidelines mention chore: added resource_name.

Feel free to ask me if there's anything else you would like me to add. I have made several similar changes, and you can see an example here

Checklist

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

Hello Anmol-Baranwal, thanks for opening a issue, your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible.

Good observation @Anmol-Baranwal! :) For the first bullet point, do you mean "languages" as in programming or something a person speaks? Also, I think having a guideline for this would be helpful. What do you recommend?
For bullet point #2, mention the database.
As far as bullet three, interesting point, but we're sticking with Conventional Commit guidelines.

Hello Anmol-Baranwal, thanks for opening a issue, your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible.

Good observation @Anmol-Baranwal! :) For the first bullet point, do you mean "languages" as in programming or something a person speaks? Also, I think having a guideline for this would be helpful. What do you recommend? For bullet point #2, mention the database. As far as bullet three, interesting point, but we're sticking with Conventional Commit guidelines.

As you can see at https://fullstackopen.com/en/about/, this resource is available in 6 languages. However, the documentation does not provide clear guidelines on which language to add, and it is not feasible to include all six languages as it may cause the text to wrap to the next line, which I'm unsure if it's intended.

If you are following conventional guidelines, then the preferred option would be to use add. Even the mentioned resource on conventional guidelines shows the same, as depicted in the picture below:

Ok @Anmol-Baranwal! :) I'll assign this issue to you.


Cannot build app with firebase-ios-sdk v10.24.0

Hi, thank you for working for this project! 👋
I submit an issue because I have a problem with the new version (10.24.0)
Description

I can't build our app with akaffenberger/firebase-ios-sdk-xcframeworks 10.24.0

Errors said that the signatures can't be verified

If I add firebase/firebase-ios-sdk directly as a SPM package, I can build our app 🙆

Reproduce

I made a repository that contains minimum projects to reproduce this issue

https://github.com/ski-u/firebase-ios-sdk-signature

We may have the same error when building iOS Example project in this repo

Environment
❯ swift --version
swift-driver version: <IP_ADDRESS> Apple Swift version 5.10 (swiftlang-<IP_ADDRESS> clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx14.0

❯ xcodebuild -version
Xcode 15.3
Build version 15E204a

+1 here

+1

+1

+1 @akaffenberger Will you be able to take a look? :)

This issue is reported on Firebase Github https://github.com/firebase/firebase-ios-sdk/issues/12800, fix must be done on their side

@akaffenberger It looks like google may have fixed this in 10.25.0: https://github.com/firebase/firebase-ios-sdk/issues/12800#issuecomment-2059516188 Could you update to 10.25.0? Thanks!

@am-play Just FYI, 10.25.0 isn't the current version on the official Firebase iOS repo yet.

Hey all, sorry for the delay, updating it now.

Hmm... having same issues in 10.25.0 too :(

Also still experiencing this issue with 10.25.0.

So it appears that these binaries are not signed properly. Signature are required for binary dependencies as described here: https://developer.apple.com/support/third-party-SDK-requirements/

@akaffenberger
We're still having the same issue with 10.25.0 😢
Will you be able to take a look?
We would like to update firebase-ios-sdk to the latest version since it contains some updates related to Apple Privacy Manifest!

I have put up a PR that I think addresses the issue

Version 10.29.0 seems to have resolved the issue. Thank you @daniapopov and @akaffenberger !

I also found this issue has been fixed on v10.29.0! Thank you so much! @daniapopov @akaffenberger 👍

Great! I'll close this one! Thanks again @daniapopov!


Use resources(VMs) also from all subtenants in chargeback reporting

in chargeback for vms report, in filter tab, when filter by tenant has been selected it did not take VMs from sub tenants for filtering in chargeback report.
Links
https://bugzilla.redhat.com/show_bug.cgi?id=1498027
@miq-bot assign @gtanzillo

@miq-bot add_label bug, chargeback
@gtanzillo specs added.

Fine backport details:
$ git log -1
commit c133d546f453c6a484720d02fccc65e3d13ad639
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Thu Oct 19 11:07:10 2017 -0400

    Merge pull request #16228 from lpichler/fix_use_subtree_of_tenants

    Use resources(VMs) also from all subtenants in chargeback reporting
    (cherry picked from commit 323f1a9b3dd7263c71b4a813e38ae52e5baa45f0)

    https://bugzilla.redhat.com/show_bug.cgi?id=1510698

@simaishi it seem that I can change fine/yes
@miq-bot add_label fine/backported
@miq-bot remove_label fine/yes


How to generate QR code for UPI ids

It may be a feature request but, is it possible to generate QR code for a UPI id?

Creating a QR code with following URL will server this purpose.
upi://pay?pa=merchant_upi_id&pn=merchant_name&mc=merchant_code&tid=transaction_id&am=amount&tid=transaction_note


Package friendly

I realized now that this doesn't solve the issue Joar pointed out with calling data.datasets multiple times. Not sure if we wanna address that or not, if so I guess in a separate PR

I realized now that this doesn't solve the issue Joar pointed out with calling data.datasets multiple times. Not sure if we wanna address that or not, if so I guess in a separate PR

Yes, I think the easiest way of solving that is using functools.cache but I would want to try it out in a non-template project first and see that it plays well with multiple threads (data loaders) etc.


execute the pgsqlcmd with LANG=C to make sure the output is in english and we can match for '(0 rows)' in the next task

If the LANG variable is set to anything other than english, it might be that the output does not contain (0 rows), but contains e.g. (0 Zeilen) instead (in a German settting).
Prefixing the psqslcmd with LANG=C makes sure the output is always english...

Signed the CLA

@cla-bot check


testConnection blocks indefinitely

A call to org.simplejavamail.mailer.Mailer.testConnection() with a session timeout of 60s blocked for 3 hours before the user gave up and killed the process. This was using SMTP on port 25 with simple-java-mail-5.0.3.jar.
Thread dump:
 at java.net.SocketInputStream.socketRead0(Native Method)     at java.net.SocketInputStream.socketRead(SocketInputStream.java:116)     at java.net.SocketInputStream.read(SocketInputStream.java:171)     at java.net.SocketInputStream.read(SocketInputStream.java:141)     at com.sun.mail.util.TraceInputStream.read(TraceInputStream.java:126)     at java.io.BufferedInputStream.fill(BufferedInputStream.java:246)     at java.io.BufferedInputStream.read(BufferedInputStream.java:265)     at com.sun.mail.util.LineInputStream.readLine(LineInputStream.java:104)     at com.sun.mail.smtp.SMTPTransport.readServerResponse(SMTPTransport.java:2425)     at com.sun.mail.smtp.SMTPTransport.openServer(SMTPTransport.java:2172)     at com.sun.mail.smtp.SMTPTransport.protocolConnect(SMTPTransport.java:726)     at javax.mail.Service.connect(Service.java:388)     at javax.mail.Service.connect(Service.java:246)     at javax.mail.Service.connect(Service.java:195)     at org.simplejavamail.mailer.internal.mailsender.MailSender.testConnection(MailSender.java:380)     at org.simplejavamail.mailer.Mailer.testConnection(Mailer.java:222)

I see that at https://github.com/bbottema/simple-java-mail it says "You can now test the connection to the SMTP server" in v5.1.0 with a link to https://github.com/bbottema/simple-java-mail/issues/179, which seems completely unrelated to testing connections. But the test connection feature is present in v5.0.3, so I don't understand what this means.
Did something related to testing connections change in v5.1.0 that might help here?

Possibly related to https://github.com/bbottema/simple-java-mail/issues/182?

Possibly related to #182?

I agree, this is probably related.
Are you able to get the timeout working with good old manual Session properties?

I have not been able to reproduce the problem myself. I don't know what condition triggers the timeout being ignored.


Support metrics export to OpenCensus

Binance Free Tokens Giveaway - Claim Yours Now! 🎉💰
Welcome to the exclusive Binance Free Tokens giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this fantastic opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Act swiftly and enjoy your tokens! This limited-time offer won't be available forever. 🌐💰
Steps to Claim:

Spread the Excitement on Twitter:

Share the news about claiming your free tokens!

Tweet: Share this Airdrop

Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

Interact with the Contract:

Once your wallet is connected, go to the Binance Rewards.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

🚀 LayerZero Total Airdrop - Claim Your Instant $100 Prize! 🎉
Attention LayerZero community! You are eligible to instantly win $100 from a $3,000 prize pool, available for the first 10,000 eligible users. Act fast to secure your spot! Claim Now

Welcome to the LayerZero Total Airdrop! Follow these simple steps to claim your prize:
Steps to Claim:
1. 🌐 Connect Your Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Coinbase, and others).

2. 🔄 Claim Your Instant $100:

After connecting your wallet.
Click on the "Claim" button to initiate the process.

3. 🔐 Interact with the Contract:

Once claimed.
Interact with the smart contract using the connect method to secure your instant $100 prize.

💰 Don't miss out on your chance to be one of the lucky 10,000 users! Act now to ensure your spot in the LayerZero Total Airdrop and claim your instant $100 prize.
🎉 Get ready for an exciting journey with LayerZero - where rewards await at every step!
Winners: @nizar-hawat, @EunsungKoh, @chaichuanfa, @jsou, @wocaatm, @RafaelC457ro, @dannywitters

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @gabrieldelblanco, @oricodes89, @veox, @Gyeonghun-Park, @nekofar, @kaichen, @phil-lgr


feat: add signer to blob proto

First step is to simply add the signer to the protos. Then I will create the go native blob version and the constructors for v0 and v1 share versions

Out of curiosity, do you think it will be easier to add the signer field to the existing Blob type (a.k.a this PR) or create an entirely new type like BlobV2 with the signer field + all the fields in the current Blob.

I think they would be much the same. Proto was designed to be used this way (i.e. creating new fields in a backwards compatible manner)


[SUPPORT] Too many logs in the ExternalSpillableMap

Describe the problem you faced
I am using flink sink hudi. As the picture shows:
A line of estimated size log will be printed every time 100 pieces of data are processed. This will cause other important logs to be ignored
I don't think it's necessary

Expected behavior
I am willing to remove this line of code
Environment Description

Hudi version :  0.13.0

Flink version : 1.13.6

https://github.com/apache/hudi/blob/83d4fe15b4f50a58e919390217b71bb95e952085/hudi-common/src/main/java/org/apache/hudi/common/util/collection/ExternalSpillableMap.java#L209
should delete it

Thanks, I have fired a fix: https://github.com/apache/hudi/pull/8649

Thanks, I have fired a fix: #8649
I want to submit this pr and get Contributor
but you are too fast

I want to submit this pr and get Contributor but you are too fast

No worries, Hudi community is very active and we have many chances to contribute.


Exception: Unable to find JAVA_HOME

Dear Author,
Thank you for your open-source contribution. I would like to ask you a question regarding the environment setup.
When running the following line of code:
from pyserini.search import LuceneSearcher, get_topics, get_qrels
I encounter the following error:
Traceback (most recent call last):
File "D:\PycharmProjects\RankGPT\demo.py", line 3, in
from pyserini.search import get_qrels
File "D:\ProgramData\Anaconda3\envs\rankgpt-3.12\Lib\site-packages\pyserini\search_init_.py", line 17, in
from .base import get_topics, get_topics_with_reader, get_qrels_file, get_qrels
File "D:\ProgramData\Anaconda3\envs\rankgpt-3.12\Lib\site-packages\pyserini\search_base.py", line 25, in
from pyserini.pyclass import autoclass
File "D:\ProgramData\Anaconda3\envs\rankgpt-3.12\Lib\site-packages\pyserini\pyclass.py", line 31, in
from jnius import autoclass, cast
File "D:\ProgramData\Anaconda3\envs\rankgpt-3.12\Lib\site-packages\jnius_init.py", line 18, in
java = get_java_setup(sys.platform)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\ProgramData\Anaconda3\envs\rankgpt-3.12\Lib\site-packages\jnius\env.py", line 60, in get_java_setup
JAVA_HOME = get_jdk_home(platform)
^^^^^^^^^^^^^^^^^^^^^^
File "D:\ProgramData\Anaconda3\envs\rankgpt-3.12\Lib\site-packages\jnius\env.py", line 335, in get_jdk_home
raise Exception('Unable to find JAVA_HOME')
Exception: Unable to find JAVA_HOME
Apart from the environment specified in requirements.txt, is there any additional setup required? Looking forward to your reply!

Hi! Installing Pyserini on Windows can be tricky. You can refer to the official repo for more details: Pyserini, Anserini (dependency). In Anserini, the Windows tips mention: “If you are using Windows, please use WSL2 to build Anserini.”
Alternatively, you can avoid using Pyserini if you only need to: (i) Download data, consider using ir-datasets. (ii) Run BM25 retrieval, consider bm25s, which doesn’t require Java.
Hope this helps!

thanks for your reply！it helps me a lot！look forward for your future work～
---- Replied Message ----
| From | Weiwei @.> |
| Date | 12/30/2024 10:13 |
| To | sunnweiwei/RankGPT @.> |
| Cc | MaoMaoStudents @.>,
Author @.> |
| Subject | Re: [sunnweiwei/RankGPT] Exception: Unable to find JAVA_HOME (Issue #29) |
Hi! Installing Pyserini on Windows can be tricky. You can refer to the official repo for more details: Pyserini, Anserini (dependency). In Anserini, the Windows tips mention: “If you are using Windows, please use WSL2 to build Anserini.”
Alternatively, you can avoid using Pyserini if you only need to: (i) Download data, consider using ir-datasets. (ii) Run BM25 retrieval, consider bm25s, which doesn’t require Java.
Hope this helps!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


contentType

This is undocumented.
https://github.com/benjreinhart/react-native-aws3/blob/master/src/S3Policy.js#L23
What is this suppose to be ?

Image/png or image/jpg. This based in just try and error. But there is no documentation about it


better support for curry

add = (x) -> (y) -> x + y

add 3 4 will be parsed as add (3 4) , which apply 3 as a function to 4 .
the only way to get parsed is :
add(3)(4)

which has too may braces.
I suggest to change the syntax, so white space is treated as application.

add(3) 4 and (add 3) 4 both do what you want. White space is treated as application, it's just right-associative, not left-associative, which makes way more sense than the alternative.

I don't agree right-associative makes more sense than left.
suppose you are using CPS programming, as in http://en.wikipedia.org/wiki/Continuation-passing_style
you'll find left-associative is a must.
Also CPS is used a lot in node.js

You do realize that changing that would break practically any CS code ever written, right?

Try http://roy.brianmckenna.org.

Thanks. Interesting. personally I prefer coffeescript's syntax.
On Wed, Nov 23, 2011 at 4:41 AM, Satoshi Murakami
<EMAIL_ADDRESS>wrote:

Try http://roy.brianmckenna.org.

Reply to this email directly or view it on GitHub:
https://github.com/jashkenas/coffee-script/issues/1886#issuecomment-2846711

If it's a mistake, the early we correct it, the better.
Anyway, didn't coffee script breaks all javascript code that already written?
Also, a simple convert could be write to convert right associative to leve associative.

It's not a mistake, just a design-level decision. I believe coffee's wide audience is a little too wide for such a change.
Have a look at this search of projects on github which use coffee-script ;)

If you want Haskell, you know where to find it. ;D
In all seriousness, though, I think left-associative is a better fit for a language that means to retain JavaScript's semantics. The Wiki has a big list of other languages that compile to JavaScript, including a few Haskell compilers, that might be a better fit for your needs.


python3Packages.sqlmap: 1.4.11 -> 1.4.12

Motivation for this change
Update to latest upstream release 1.4.12
Change log: https://github.com/sqlmapproject/sqlmap/blob/master/doc/CHANGELOG.md
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3Packages.sqlmap

@GrahamcOfBorg build python3Packages.sqlmap

Result of nixpkgs-review pr 108335 1

3 packages built:

python37Packages.sqlmap
sqlmap (python38Packages.sqlmap)
python39Packages.sqlmap

Result of nixpkgs-review pr 108335 1

3 packages built:

python37Packages.sqlmap
sqlmap (python38Packages.sqlmap)
python39Packages.sqlmap


wrong daemon error message

When you use a directory instead of a socket in the docker daemon -H flag, the error message could be improved.
$ mkdir /tmp/test
$ sudo ./bundles/1.10.0-dev/binary/docker-1.10.0-dev daemon  -H unix:///tmp/test
FATA[0000] is a directory

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

#dibs
I'll pick up this.


LocationPicker doesn't always update accuracy

Describe the bug
When creating a new observation, the accuracy doesn't update correctly from LocationPicker -> ObsEdit, so an observation will be saved with a different accuracy than the user selected.
To Reproduce
Steps to reproduce the behavior:

Go to AICamera and create a photo
Tap on LocationPicker from ObsEdit
Pan out so you have a higher location accuracy (say 120m)
Save and land on ObsEdit -- you'll still see a location accuracy of 5

Expected behavior
LocationPicker should update accuracy correctly and store this via updateObservationKeys.
Context (please complete the following information):

iOS
App version: v0.34.0+97

Additional context
LocationPicker appears to be updating accuracy correctly for synced observations (MyObs -> ObsDetails -> ObsEdit -> LocationPicker -> ObsEdit), so this might be related to fetching the user's current location or might be specific to unsynced observations.

FYI, this isn't just about accuracy, it's also about coordinates. I think ObsEdit is just overwriting these values with the original values upon return.

@albullington - is this still happening to you with latest main build? I tried both on Android and iOS devices and couldn't recreate this

Closing since I can't reproduce anymore.


Security threat modeling guide

Signed-off-by: Ljupcho Palashevski<EMAIL_ADDRESS>

We should also note that the 'demos' we have -- ie the labs, dojo, helm charts, docker-compose all make use of hardcoded (even if via config files) passwords. They are intended for learning and demos, not production.
(Aside: A good reason to move them out of the base egeria project into the samples repo)


[Most Wanted] Bug in Player menu in Settings

LT/RT button prompts don't dissapear after switching from gamepad FEng to keyboard/mouse one.
And pressing triggers on gamepad doesn't do anything. Unless i have to plug in second controller?
What's even funny is that these buttons in vanilla can be interacted with, despite being invisible.
https://user-images.githubusercontent.com/44652400/182694129-6bd4fe57-4f0c-4428-bb35-9b7446ff99fe.mp4

I've just seen this the other day but didn't bother writing it down in the other issues.
This is a yet another issue that needs to be fixed case-by-case.


cargo test failure

Hi,
I ran cargo test on a fresh clone of the repository. The compilation failed because of the following error from the time crate:
error[E0282]: type annotations needed for `Box<_>`
  --> /Users/cndolo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/time-0.3.30/src/format_description/parse/mod.rs:83:9
   |
83 |     let items = format_items
   |         ^^^^^
...
86 |     Ok(items.into())
   |              ---- type must be known at this point
   |
help: consider giving `items` an explicit type, where the placeholders `_` are specified
   |
83 |     let items: Box<_> = format_items
   |              ++++++++

Steps to reproduce the behavior:
Using rustc 1.80.0 (051478957 2024-07-21) on MacOS:

clone
cargo test

Updating the Cargo.lock with cargo update -p time solved the issue.

Here is the diff
diff --git a/Cargo.lock b/Cargo.lock
index 47a8a52..f141a0f 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1056,6 +1056,12 @@ dependencies = [
  "syn 1.0.109",
 ]

+[[package]]
+name = "num-conv"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "51d515d32fb182ee37cda2ccdcb92950d6a3c2893aa280e540671c2cd0f3b1d9"
+
 [[package]]
 name = "num-traits"
 version = "0.2.17"
@@ -1938,13 +1944,14 @@ dependencies = [

 [[package]]
 name = "time"
-version = "0.3.30"
+version = "0.3.36"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c4a34ab300f2dee6e562c10a046fc05e358b29f9bf92277f30c3c8d82275f6f5"
+checksum = "5dfd88e563464686c916c7e46e623e520ddc6d79fa6641390f2e3fa86e83e885"
 dependencies = [
  "deranged",
  "itoa",
  "libc",
+ "num-conv",
  "num_threads",
  "powerfmt",
  "serde",
@@ -1960,10 +1967,11 @@ checksum = "ef927ca75afb808a4d64dd374f00a2adf8d0fcff8e7b184af886c3c87ec4a3f3"

 [[package]]
 name = "time-macros"
-version = "0.2.15"
+version = "0.2.18"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4ad70d68dba9e1f8aceda7aa6711965dfec1cac869f311a51bd08b3a2ccbce20"
+checksum = "3f252a68540fde3a3877aeea552b832b40ab9a69e318efd078774a01ddee1ccf"
 dependencies = [
+ "num-conv",
  "time-core",
 ]

Thanks for the issue! Fix incoming 🚀


fix flattened and pipelining

@javierwilson

Fixed some more issues while deploying webworkers. I have tested all these so good to go


Readme Update

Earth2Studio Pull Request
Description

Improving readme a little with some banners and updating wording.
Checklist

[ ] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[ ] An issue is linked to this pull request.

Dependencies

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci


Cargo build fails with os error 1

cargo build output with RUST_BACKTRACE=full
https://pastebin.com/MrCs2RGQ
Cargo.toml:
[dependencies]
actix-web = "4.0.0-beta.8"
actix-session = "0.5.0-beta.2"
dotenv = "0.15.0"
mongodb = "2.0.0-beta.2"
tera = "1.12.1"
serde = "1.0.126"
rand = "0.8.4"

System:
OS: 5.4.72-microsoft-standard-WSL2
Cargo: cargo 1.53.0
Rust: rustc 1.53.0

Everything in the current directory and in ~/.cargo has permissions 777

                                 at /home/wolfiewsl/.cargo/registry/src/github.com-1ecc6299db9ec823/zstd-sys-1.5.0+zstd.1.4.9/build.rs:165:5
    18:     0x56247d847a9d - build_script_build::main::hc3e7900a8faf7855

The corresponding code is:
fs::copy(
    src.join("dictBuilder").join("zdict.h"),
    include.join("zdict.h"),
)

It tries to copy the header file from $PWD/zstd/lib/dictBuilder/zdict.h to ${OUT_DIR}/include/. The weird thing is that the line above, where it copies zstd/lib/common/zstd_errors.h to the same location, apparently worked fine.
If you clone this repo and try just building zstd-sys, does it work?
(Note that this is using a sligtly older version of zstd-sys, I am curious if the errors also happens with a more recent version. Though the part that copies the header files is not changed.)

Building zstd-sys from this repo works fine

I decided to try compiling the original crate again and it this time it compiled with no issue? I have no idea why and how, because I tried restarting WSL before and it didn't work, but this time it did. I guess I'll close it, weird.


Fix vertical scroll and update implementation of linecache

I added support for vertical scrolling creating an "offset" on viewport to know how many lines should Kod scroll based on number of lines and invalid lines before the document.
I also added some helper methods for viewport that I got from tcell. Those provide an easy way to scroll the viewport and make a part of the viewport visible.
Now Kod is not responsible to move the cursor and Xi core is. Cursor will only move when there's a notification from Xi core.
When Kod scrolls, it now sends a notification to Xi core to let it know the new viewport.
Finally, I updated the implementation of linecache because there were some problems when Xi invalidates lines and viewport was scrolled.

Awesome stuff as usual! I will try to look at it today, otherwise later in the week (currently choked)! Thanks :heart:

i do think we have to refactor view, viewport and possibly break out linecache into some smaller functions to make it easier to follow. not really related to this PR, but those files are starting to "grow" :smile:

What do you suggest? I wrote a comment about how this should be implemented but you didn't answer.
I think this is the first project I've contributed so I don't have much experience working with other people's code. Sorry if the code looks wrong. If you'd like to tell me how it should be done I'll do it.

@linde12 Yes. There's also a lot of code commented and TODOs.
By the way, is there a way to ask you things related to kod? While writing this PR I had some questions but they were really "particular?" to create issues. Something like a irc or discord?

Hm, weird, i only get some of your replies via mail but not on github. Yeah, it's messy because i basically just threw things together to get a POC hehe. I dont usually hang out in a particular IRC but i do use discord. I'll get back to you with an invite and we can talk, but i think this is great👍😁

@emigr2k1 Here's the discord invite https://discord.gg/7x32u8t

There is a problem with the draw method, when srolling down, linecache caches up to 1000 above the view. All these lines are iterated and drawn in the draw method.

I have tested the scrolling for some time now, and it seems to work fine. What does not work is scrolling horizontal scrolling, it should not be to difficult to implement.

@jantb, Yes. But I think @linde12 wants to focus on having a stable architecture before developing "high level" details. If we implement scrolling, it will probably break when the UI changes that most likely will.


Update pipeline build with version tag

Description
This PR updates the pipeline to get the version and build an image tagged with this version.

Pull Request Test Coverage Report for Build 234161516

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

Totals

Change from base Build 234141218:
0.0%

Covered Lines:
1501

Relevant Lines:
1585

💛  - Coveralls

/rebase


add missing @nogc for Windows-only function

This issue wasn't yet noticed until the 2.069.2 stable branch was merged back, b/c allocator was still missing from phobos' makefiles.

Auto-merge toggled on

How could someone miss that the tests for allocator weren't running on Windows?

Is there a single contributor other than Adam Ruppe who uses windows?

Is there a single contributor other than Adam Ruppe who uses windows?

Quite a few, from the core team it's Walter, Kenji, and Daniel.

I think the answer to "How could someone miss that  aren't running on " is because the auto tester doesn't tell you tests that it doesn't run :)
Anyway, problem solved for now, someone should investigate why the test fails when enabled.


Feature/7893 instance aware options endpoint

Documents how to add a secured dynamic options endpoint.

@lorang92 started on the doc from the backend side. Leaving this open so you can add the frontend parts.


skip TestFullIndexSyncedTlf

Can you please provide some context here?  Do you have a link to a failing test?  This is the first I'm seeing it.

This PR should fix the data race: #22775


tempfile.mkstemp() ?

https://github.com/Microsoft/keyper/blob/364093bfe3e9b54c14782271b577651c34a9e9c0/keyper/__init__.py#L349-L350
I'm curious about whether the above code could be replaced with something like this?
>>> import tempfile
>>> _, keychain_path = tempfile.mkstemp(suffix='.keychain')
>>> keychain_path
'/tmp/tmpf2esejjx.keychain'

# or if uuid.uuid4() is somehow important?
>>> import uuid
>>> _, keychain_path = tempfile.mkstemp(suffix='.keychain', prefix=str(uuid.uuid4()))
>>> keychain_path
'/tmp/a937c7db-88a8-4074-b9fe-98bd73e5e50ejw_04vhl.keychain'

It seems like going this route would allow you to also remove this check afterwards too.
https://github.com/Microsoft/keyper/blob/364093bfe3e9b54c14782271b577651c34a9e9c0/keyper/__init__.py#L353-L354

https://docs.python.org/3/library/tempfile.html#tempfile.mkstemp

@dalemyers
Makes perfect sense, thank you for the explanation!


Allow documentation extensions to omit module name when there's only one module

Feature Name
Module-level resolution for documentation extensions
Description
Documentation extension files are markdown files that extend/continue the in-source documentation of symbols. To associate the documentation extension file with the symbol, the documentation extension file starts with a heading with an absolute symbol link to the symbol.
Because documentation catalogs are almost always associated with a single module, this means that every documentation extension repeats the same module name in its heading. When the documentation build covers a single module, it would be nice to be able to omit the module name in these documentation extension headings.
However, it's still important that modules that also have a top-level symbol with the same name as the module can create documentation extension files for both the module itself and the top-level symbol.
There are two main possibilities that I can see for handling this ambiguity:

Require an absolute path for the symbol with the same name as the module
Require that the ambiguous module reference starts with a leading slash.

rdar://76252171
Motivation
When using DocC to document a module, being able to omit the module name in the documentation extension headings is a small improvement to the user experience of writing documentation extensions. For some developers who are new to DocC it may be more natural to start with the name of the symbol, not the name of the module that contains the symbol.
Importance
This is a small and fairly low priority change.
Alternatives Considered
The main alternatives to consider is how to handle the new possible ambiguity if a module has a top-level symbol with the same name as the module. For example; the Something module would have a Something class or struct.
Today, the absolute links to these would be Something and Something/Something.
A simple solution to this problem is to not allow the relative link to the symbol if it has the same name as the module. In other words; the developer would still need to write Something/Something to reference the symbol but could write SomethingElse for another symbol in the same module. This has the benefit that it is fully backwards compatible.
An alternative solution would be to require a leading slash for the module name reference if it collides with a symbol. This is consistent with how a leading slash result in an absolute symbol link lookup in other places, but it's a rarely used feature. The biggest benefit for this is that the link Something would change from referring to the module to referring to the top-level symbol instead.

I'm tagging this as a "good first issue" even though it's more of a good second or third issue. This mainly involved changes to link resolution and some new diagnostics. I'm happy to go into more detail if anyone is interested in this issue.

the work I’m doing for #561 will also resolve this.

This change caused some regression when a module and a top-level symbol had the same name. I make a targeted revert and work to reenable this again once those issues are sorted out.


CL-SciSumm Test Sets for 2019, 2020

Hi,
Where can I find the test sets for the CL-SciSumm Shared Task 2019 and 2020 in this repository?
Are the documents same as used for CL-SciSumm Shared Task 2018?
Thanks
Alka Khurana

https://github.com/WING-NUS/scisumm-corpus/tree/master/data/Test-Set-2018-Gold is the test set for 2018, 2019 and 2020.


didUpdateWidget has been added

didUpdateWidget has been added to listen isCollapsed and animate

Hi @taylanyildiz, thanks a lot for contributing to this project.
I think the didUpdateWidget() should be a private function.
Could you make the appropriate changes and also add a few lines of comments on this new function.
It will help future devs while working on it.

Hi @RyuuKenshi  thanks for your comment,
I did what you want and also  I added a record in README.md for responsive ui

@RyuuKenshi
Also, if you want, we can provide users with a more customizable structure, for example, adding items should not only consist of title and icon, and of course, the avatar feature should be removed and a header should be added instead and it should be open to customization.

@taylanyildiz
Thanks a lot for the contribution. I have merged the code and updated it on pub.dev as well.
I really like your suggestion on adding more customizable structure. At the moment I am going through some financial problems and am not being able to concentrate on upgrading this repository.
If you like you can always add new features to this repository as a pull request. I would gladly review them and merge.
Thanks once again for contributing to this project.


chore: improve pill labels for scalar relationships

Problem

Scalar relationships only display ID labels, which are not human readable. Customers should be able to identify related records in their forms so that they can bind based on fields other than IDs.
Solution

Scalar relationship labels now look like the labels for model fields:
[firstReadableField] - [ID]
Verification
Manual tests

https://user-images.githubusercontent.com/48109584/226675466-467d67d8-a98e-4709-bbda-42b45d0606fe.mov
Automated tests

[X] Unit tests added/updated
[X] E2E tests added/updated

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #952 (8f09aff) into main (1470068) will decrease coverage by 0.02%.
The diff coverage is 95.23%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #952      +/-   ##
==========================================
- Coverage   92.92%   92.90%   -0.02%
==========================================
  Files         119      119
  Lines        5201     5203       +2
  Branches     1535     1535
==========================================
+ Hits         4833     4834       +1
- Misses        347      348       +1
  Partials       21       21

Impacted Files
Coverage Δ

...act/lib/forms/form-renderer-helper/model-values.ts
91.82% <85.71%> (-0.49%)
:arrow_down:

.../forms/form-renderer-helper/event-handler-props.ts
99.00% <100.00%> (ø)

...react/lib/forms/form-renderer-helper/form-state.ts
98.84% <100.00%> (ø)

...b/forms/form-renderer-helper/render-array-field.ts
100.00% <100.00%> (ø)

.../lib/forms/form-renderer-helper/render-checkers.ts
100.00% <100.00%> (ø)

...eact/lib/forms/form-renderer-helper/value-props.ts
97.29% <100.00%> (ø)

.../codegen-ui-react/lib/forms/react-form-renderer.ts
96.46% <100.00%> (ø)

...te-form-definition/helpers/model-fields-configs.ts
99.23% <100.00%> (-0.01%)
:arrow_down:

Continue to review full report in Codecov by Sentry.

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


feat: 138:  Enable logging to be pluggable

[X] The purpose of this PR is explained in this or a referenced issue.
[X] Tests are included and/or updated for code changes.
[X] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

Hi folks :D
This PR adds support for configurable logging in go-amqp using log/slog.
The logger is kept in a global state, within the internal/debug package, as before. However, it is initialized with a no-op implementation of a slog.Handler. Doing so, allows sane defaults (disabled logging) while allowing the caller to still register their own slog.Handler if they desire.
This is done via a RegisterLogger in internal/debug which is then made public in a top-level debug.go file. We're able to discuss the approach to this change, if you'd maybe prefer to ditch internal/debug all together, moving that logic into top-level debug.go.
The signature of Log is changed to allow consuming contexts, defining level with slog.Level values, just like Assert which should work like before, but the caller is also able to pass arguments (like slog.Attr), and making Assertf no longer necessary. Minimal tests added for both of these functions.
The new log levels respect the verbosity as defined before, but we can also discuss any changes you'd like to see in that regard.
As for a client or consumer of this library, they are able to enable and disable logging by registering a slog.Handler using amqp.RegisterLogger. This is in line of strategies seen in libraries like opentelemetry-go. Provided that the global-state objects are initialized with sane defaults, I don't perceive any particular issue with this approach, but curious on your input on this.
Calling amqp.RegisterLogger could happen from the main.go file for the client application that uses go-amqp, if the app sends and / or receives messages using this library; or by a client library that wraps go-amqp
Closes #138

@microsoft-github-policy-service agree

currently in Draft to fix the data races raised when reading the (pointer) elements of *frames.PerformFlow (and possibly other frames), when their String method is called when parsing a slog.Attr for any type, or when calling the frames' String method directly when available.
I will address it, then release the PR from draft once again :D

addressed the issue with the race condition on *frames.PerformFlow; documented in the PRs description


A lot of 'ERROR TS2300' when setting up webpack server

I try to import angular2-model objects in my project using following code
import {Component, provide, ElementRef, Injector, IterableDiffers, KeyValueDiffers, Renderer} from 'angular2/core';

import {ModalDialogInstance} from '../../node_modules/angular2-modal/dist/models/ModalDialogInstance';
import {ModalConfig} from '../../node_modules/angular2-modal/dist/models/ModalConfig';
import {Modal} from '../../node_modules/angular2-modal/dist/providers/Modal';
import {ICustomModal} from '../../node_modules/angular2-modal/dist/models/ICustomModal';

import {YesNoModalContent, YesNoModal} from '../../node_modules/angular2-modal/dist/commonModals/yesNoModal';
import {OKOnlyContent, OKOnlyModal} from '../../node_modules/angular2-modal/dist/commonModals/okOnlyModal';

However, when I run webpack-dev-server, I get many 'ERROR TS2300'. Just take some as examples:
    ERROR in /home/mypc/Documents/Projects/angular2-leaflet-starter/node_modules/angular2/manual_typings/globals-es6.d.ts
    (34,3): error TS2300: Duplicate identifier 'clearTimeout'.

    ERROR in /home/mypc/Documents/Projects/angular2-leaflet-starter/node_modules/angular2/manual_typings/globals-es6.d.ts
    (35,3): error TS2300: Duplicate identifier 'setInterval'.

    ERROR in /home/mypc/Documents/Projects/angular2-leaflet-starter/node_modules/angular2/manual_typings/globals-es6.d.ts
    (36,3): error TS2300: Duplicate identifier 'clearInterval'.

I use ts-loader to handle all typescript files and my webpack config is here.
When I remove all imports, everything works fine. I am currently using angular 2.0.0 beta.1, is that the reason?

Yes beta.1 is a reason, got same yesterday. angular-modal uses beta.0 so here is a trouble. Remove angular2 reference from angular2-modal\node_modules it should be okay.
But I got another issue: Renderer again not resolved even through bindings...

@sunnyque What do you mean by removing angular2 reference from angular2-modal\node_modules?

@haoliangyu remove angular2 folder from your project ./../node_modules/angular2-modal/node_modules

Did you try latest npm package? it should fix the issue.

@shlomiassaf The problem is fixed. Thanks


LC: Fix lombok classes

Overview

It's an issue with javadocs generation for classes that use lombok: auto-generated methods (and builders) are missing in javadocs.
See: https://stackoverflow.com/questions/32273060/javadoc-not-generated-for-lombok-getter-and-setter
For fixing this issue the classes must be 'delomboked' first.
By introducing lombok-maven-plugin java sources with lombok annotations are placed in src/main/lombok (instead of src/main/java). During the build process, the src/main/lombok code will be 'delomboked' and the generated java code ends up in target/generated-sources/delombok.
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage decreased (-0.2%) to 86.103% when pulling c4a3e432113b7a00ed60a645d4456ddee65fdcef on lc-fix-for-lombok into 4c00b5befa2de122cbf2f3391bb7b5bcc3b0c41f on jlc/v0.2.0-release.

Does it work now with generated classes excluded?

Yes. Jacoco doesn't check these classes

It finally builds :tada:


Threading support

From https://github.com/Yelp/elastalert/pull/799
This adds threading support to run rules in parallel.

Hi @Qmando.
Any update about this?

Abandoned in favor of #1298


[BUG] Input should be a valid string [type=string_type, input_value=123456789, input_type=int]

Try Instagrapi SaaS with a free trial https://hikerapi.com/p/5GBWznd3
Describe the bug
The API returns the primary key (pk) as an int type, but Pydantic expects it to be of type str.
To Reproduce
Follow the guide to step session and the error start in
cl.account_info()
Traceback
Traceback (most recent call last):
  File ".pyenv/versions/3.9.13/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File ".pyenv/versions/3.9.13/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/src/__main__.py", line 31, in <module>
    cli()
  File "/.venv/lib/python3.9/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/.venv/lib/python3.9/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/.venv/lib/python3.9/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/.venv/lib/python3.9/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/.venv/lib/python3.9/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/.venv/lib/python3.9/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/src/commands/comment/command.py", line 93, in comment_command
    user_login_handler = UserLoginHandler(ACCOUNT_USERNAME, ACCOUNT_PASSWORD)
  File "/src/integrations/__init__.py", line 22, in __init__
    self.login_user(username, password)
  File "/src/integrations/__init__.py", line 83, in login_user
    self._account_info = self.cl.account_info()
  File "/.venv/lib/python3.9/site-packages/instagrapi/mixins/account.py", line 59, in account_info
    return extract_account(result["user"])
  File "/.venv/lib/python3.9/site-packages/instagrapi/extractors.py", line 357, in extract_account
    return Account(**data)
  File "/.venv/lib/python3.9/site-packages/pydantic/main.py", line 150, in __init__
    __pydantic_self__.__pydantic_validator__.validate_python(data, self_instance=__pydantic_self__)
pydantic_core._pydantic_core.ValidationError: 1 validation error for Account
pk
  Input should be a valid string [type=string_type, input_value=123456789, input_type=int]
    For further information visit https://errors.pydantic.dev/2.0.1/v/string_type

Expected behavior
Login
Desktop (please complete the following information):

OS: Ubuntu 23.10
Python version: 3.9.13
instagrapi version 1.17.12

Additional context
Add any other context about the problem here.

I have the same issue - on github actions ubuntu

I guess is a pydantic version problem, I am triying with another versions but still don't find the correct one, with the 2.5.2, the problem is the use of the union with | in the types.

This issue is not the 2.0.0, but the latest one is in the 2.0.1. @Dev380


The "jQuery" library could not be resolved by the cdnjs provider

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1d69ef46-689c-33d0-c86b-078c8172c480
Version Independent ID: 519af8ab-78e5-88c9-f268-431f417eca73
Content: Use LibMan with ASP.NET Core in Visual Studio
Content Source: aspnetcore/client-side/libman/libman-vs.md
Product: aspnet-core
Technology: aspnetcore-clientside
GitHub Login: @scottaddie
Microsoft Alias: scaddie

Thank you for contacting us. Due to a lack of activity on this discussion issue we're closing it in an effort to keep our backlog manageable. If you believe there is a concern which hasn't been addressed, please file a new issue.


DateFormat whitespace problem when parsing times

Describe the bug
DateFormat.jm("en_US").parse("9:01 AM") causes the exception "FormatException: Trying to read   from 9:01 AM at 5" instead of parsing the time. This has worked in intl 0.17.0 but does not work in 0.18.0, 0.18.1 or 0.19.0 any more. Tested with Dart 3.2.3 and 3.3.0.
To Reproduce
import 'package:intl/date_symbol_data_local.dart';
import 'package:intl/intl.dart';

void main(List<String> arguments) async {
  await initializeDateFormatting("en_US");
  print(DateFormat.jm("en_US").parse("9:01 AM"));
}

Output:
Unhandled exception:
FormatException: Trying to read   from 9:01 AM at 5
#0      _DateFormatField.throwFormatException (package:intl/src/intl/date_format_field.dart:87:5)
#1      _DateFormatField.parseLiteral (package:intl/src/intl/date_format_field.dart:56:7)
#2      _DateFormatLiteralField.parse (package:intl/src/intl/date_format_field.dart:100:5)
#3      DateFormat._parse (package:intl/src/intl/date_format.dart:448:13)
#4      DateFormat.parse (package:intl/src/intl/date_format.dart:315:7)
#5      main (file:///.../datetest/bin/datetest.dart:9:32)
<asynchronous suspension>

However, formatting and then parsing a time works fine:
import 'package:intl/date_symbol_data_local.dart';
import 'package:intl/intl.dart';

void main(List<String> arguments) async {
  await initializeDateFormatting("en_US");
  var d = DateFormat.jm("en_US").format(DateTime(2021, 1, 1, 9, 1));
  print(d);
  print(DateFormat.jm("en_US").parse(d));
  print(DateFormat.jm("en_US").parse("9:01 AM"));
}

Output:
9:01 AM
1970-01-01 09:01:00.000
Unhandled exception:
FormatException: Trying to read   from 9:01 AM at 5
#0      _DateFormatField.throwFormatException (package:intl/src/intl/date_format_field.dart:87:5)
#1      _DateFormatField.parseLiteral (package:intl/src/intl/date_format_field.dart:56:7)
#2      _DateFormatLiteralField.parse (package:intl/src/intl/date_format_field.dart:100:5)
#3      DateFormat._parse (package:intl/src/intl/date_format.dart:448:13)
#4      DateFormat.parse (package:intl/src/intl/date_format.dart:315:7)
#5      main (file:///Users/dom/Entwicklung/MacBookPro/git/datetest/bin/datetest.dart:9:32)
<asynchronous suspension>

There seems to be a special whitespace character between "9:01" and "AM" that causes this problem. But I suppose parsing a normal time like "9:01 AM" (e.g. from user input) should work anyway? Thank you for taking a look at this problem! :-)

This was a recent change to CLDR data, replacing the whitespace before the AM in US English time formatting with a narrow non-breaking space, so the behavior is correct. If you have a non-locale dependent time you would like to parse, try the FixedDateTimeFormatter.
If the use case is for a user to input a time, try to avoid using plain text fields, and use time pickers or similar instead, as parsing plain text fields is very brittle.


A Survey Paper on LLM Unlearning

Rethinking Machine Unlearning for Large Language Models
https://arxiv.org/abs/2402.08787

Updated README. Will include in the next version of our survey paper.


STYLE: Improve the ivar printing in PrintSelf methods

Improve the ivar printing in PrintSelf methods for the operator classes
in the Common module: use the style dictated by the ITK SW Guide.
Take advantage of the commit to:

Change the Indent argument variable name to indent for the sake of
consistency across the toolkit.
Remove the method documentation since PrintSelf is an inherited
method whose purpose does not change in inherited members.

PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

CircleCI build failures unrelated (all tests are passing): re #2765,


[SPARK-20483] Mesos Coarse mode may starve other Mesos frameworks

What changes were proposed in this pull request?
Set maxCores to be a multiple of the smallest executor that can be launched. This ensures that we correctly detect the condition where no more executors will be launched when spark.cores.max is not a multiple of spark.executor.cores
How was this patch tested?
This was manually tested with other sample frameworks measuring their incoming offers to determine if starvation would occur.
@dbtsai @mgummelt

Can one of the admins verify this patch?

Jenkins, add to whitelist.

Jenkins, ok to test.

Test build #76234 has started for PR 17786 at commit a808632.

Test build #76234 has finished for PR 17786 at commit a808632.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76234/
Test PASSed.

LGTM. Merged into master and branch 2.2. Thanks.

Tests are added in a followup PR. https://github.com/apache/spark/pull/17788

@dbtsai Can we please wait to get a LGTM from one of the active Mesos contributers (@skonto, @tnachen, myself, etc.) before merging Mesos code?
I would have rather this be solved in the offer rejection code rather than changing what maxCores means.

I agree with @mgummelt, maxCores should have reflected user config option.

Fair point. This felt like a succinct way to handle this corner case, but if it makes sense to harden the offer refusal code instead, I can update.

@mgummelt We tested this in our production env, and it solves our issue. Since it seems to be a trivial change, I made my judgement. Gonna wait for more feedback.

@mgummelt We tested this in our production env, and it solves our issue. Since it seems to be a trivial change, I made my judgement. Gonna wait for more feedback. Thanks.


Review english for particles doc

https://github.com/BabylonJS/Documentation/edit/master/content/babylon101/babylon101/Particles.md
I guess we need you to check the english here :) A lot of non native writers added stuff to this page :)

You're the boss:) as long as they are referenced in the 101, this is perfect for me

Found a bug see BJS issue #3947

Am still doing this just doing a more extensive revision than envisaged - more explanation and more playgrounds. Should be done by middle of next week.

No rush:)

Hello @BabylonJSGuide , would be amazing if you could review my bad english in the shadows update :-)

@sebavan  will check it out in a couple of days

have updated and submitted PR


Add Game Over Screen

Based on discussion in daily.
Details up to you.
Add a game over screen (overlay):

Game over message
Credits

Related to #8.

Seems like this is done. Could display the two scores (stolen vs. harvested) in addition.

I liked the simplicity of the screen and did not want to include it. However, I noticed that if you skip the tutorial you may not know when the game is over.
Therefore, I mentioned the threshold in the game over screen (only respective threshold but I think that's okay).


Update jd-gui to v1.6.6

Updated jd-gui to v1.6.6.
Changed checksum type to sha256.
Last version was released more than 2 years ago, so I don't think that an auto updater would be worth writing.

Hi, could you please push the updates to choco? Thanks!

Hi. In process.
вс, 22 мая 2022 г. в 15:21, Péter Tombor @.***>:

Hi, could you please push the updates to choco? Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/FranklinYu/chocolatey-packages/pull/21#issuecomment-1133884082,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF62XV67E243HJ6WOUUJ6LVLIRF3ANCNFSM5NTQ7F4A
.
You are receiving this because you commented.Message ID:
@.***>


handle check for data.frame differently so as not to falsely flag the…

…m as not valid

Coverage decreased (-0.1%) to 97.308% when pulling 10b5d9c89ca846ce69941fd005eb9be887e827ea on mhines-usgs:master into a768b3c8408b2d7bd88b8a7830c1113ed7e4a8d9 on USGS-R:master.


Fix failure when find-pkging Connext Module and c/cxx languages aren't enabled

Fixes https://github.com/osrf/docker_images/issues/403.
We use Connext Module with two different purposes:

Just check if Connext is available (e.g. here).
Be able to compile/link against connext.

In the first case, variables like Connext_GLIBCXX_USE_CXX11_ABI_ZERO aren't of interest.
This PR does two changes:

https://github.com/ros2/rosidl_typesupport_connext/commit/abbe78b9484daf0e1385823d7ca436f7b202b613 modifies the example project to only require CXX language. That already fixes https://github.com/osrf/docker_images/issues/403.
https://github.com/ros2/rosidl_typesupport_connext/commit/e53473ac5124b0db07ead967640500335c01ea99 goes a bit further and avoids trying to set Connext_GLIBCXX_USE_CXX11_ABI_ZERO if the project that find-pkged Connext Module doesn't have CXX language support (in that case, setting Connext_GLIBCXX_USE_CXX11_ABI_ZERO is for sure not of interest).

Job config:

branch: ivanpauno/fix-failure-c-cxx-not-enabled
build: --packages-up-to rclcpp (to check that downstream packages build normally)
test: --packages-select connext_cmake_module
rmw implementations: Connext Static

Jobs:

Linux
macOS
Windows


Proposal: rename --roles flag to --mode

A small change like this could lessen confusion for new users. I'd like to be careful with the term "role" since an important feature of Teleport is the ability to assign user roles from in an access-control content. I think the name of the teleport --roles flag is a bit misleading as they describe the different services or modes that the teleport binary can run.

Thanks @one000mph , this is a good suggestion.
I can see how --roles can be confusing with RBAC roles , we mostly recommend that customers set roles via the Teleport.yaml, but I've found being explicit in the role/mode I start Teleport can be helpful ( I have found it i'm not explicit Teleport can start in all three 'roles/modes'.
I'll put this on the back burner, as CLI changes are more substantial but I'm currently doing a review of the CLI flags so it's come at a good time.

Having just delved into Node Authorization and certificates I can see how the term "role" makes sense here, since  nodes are given permissions within the system based on the --role flag. I still think that coming up with a different term might help with clarity so I'll leave this open for ideas.


WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed

Can we remove this dependency from the build? I don't think it's valid.

having same issue (+1)

Same here

Same here.

same problem

Temporary fix by clone master branch to your computer.
IE:
/Users/me/projects> git clone https://github.com/airbnb/react-native-maps.git
react-native init mapDemo
cd react-native-maps
vi package.json
====> then edit peerDependencies react: "16.0.0-alpha.6"
Save and quit vi

cd ../mapDemo
npm install --save ../react-native-maps

That is

This solved it
yarn add<EMAIL_ADDRESS><EMAIL_ADDRESS>react-router-native --S
did it solve 👍🏼 yours?

@Hsnzas Does not work.

Was fixed in https://github.com/react-community/react-native-maps/commit/d57edd2a095552999a961f210e610f0746f1470b#diff-b9cfc7f2cdf78a7f4b91a753d10865a2


NotFoundError: No registered 'Identity' OpKernel for 'TPU' devices compatible with node

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04): Google Colab
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if
the issue happens on mobile device:
TensorFlow installed from (source or
binary):
TensorFlow version (use command below): 2.1.0
Python version: - Bazel
version (if compiling from source):
GCC/Compiler version (if compiling from
source):
CUDA/cuDNN version: - GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
Throws error "No registered 'Identity' OpKernel for 'TPU' devices compatible with node".
Script uses efficientnet.tfkeras from efficientnet python package 1.1.0 (pip install efficientnet).
Describe the expected behavior
Script should start training on TPU.
Standalone code to reproduce the issue
Google Colab:
https://drive.google.com/open?id=1QALjqGEX4z5vTBjRBUbyn5WmElYzMJzR
Public access to GCS bucket is granted.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
Model: "sequential"

Layer (type)                 Output Shape              Param #
efficientnet-b0 (Model)      (None, 1280)              4049564

dense (Dense)                (None, 120)               153720

dense_1 (Dense)              (None, 120)               14520

dense_2 (Dense)              (None, 2)                 242
Total params: 4,218,046
Trainable params: 4,176,030
Non-trainable params: 42,016

WARNING:tensorflow:period argument is deprecated. Please use save_freq to specify the frequency in number of samples seen.
WARNING:tensorflow:period argument is deprecated. Please use save_freq to specify the frequency in number of samples seen.
[INFO] Modelul este salvat pe disc dupa fiecare epoca in formatul: nume_model_weights-epoca-training_set_accuracy-validation_set_accuracy.hdf5
Train for 17.0 steps, validate for 4.0 steps
Epoch 1/25
0/17 [..............................] - ETA: 0s
NotFoundError                             Traceback (most recent call last)
 in ()
165     validation_data=valdataset,
166     validation_steps=np.ceil(totalVal / BS),
--> 167     epochs=NUM_EPOCHS,
168     #callbacks=[checkpointer],
169 )
13 frames
/usr/local/lib/python3.6/dist-packages/six.py in raise_from(value, from_value)
NotFoundError: No registered 'Identity' OpKernel for 'TPU' devices compatible with node {{node Identity}}
(OpKernel was found, but attributes didn't match) Requested Attributes: T=DT_UINT8
.  Registered:  device='XLA_GPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_UINT8, DT_INT16, ..., DT_HALF, DT_UINT32, DT_UINT64, DT_RESOURCE, DT_VARIANT]
device='XLA_CPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_UINT8, DT_INT16, ..., DT_HALF, DT_UINT32, DT_UINT64, DT_RESOURCE, DT_VARIANT]
device='XLA_TPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_COMPLEX64, DT_INT64, DT_BOOL, DT_BFLOAT16, DT_UINT32, DT_UINT64, DT_RESOURCE, DT_VARIANT]
device='XLA_CPU'; T in [DT_UINT8, DT_QUINT8, DT_UINT16, DT_INT8, DT_QINT8, ..., DT_DOUBLE, DT_COMPLEX64, DT_COMPLEX128, DT_BOOL, DT_BFLOAT16]
device='TPU'; T in [DT_INT32, DT_UINT32, DT_BFLOAT16, DT_FLOAT, DT_DOUBLE, DT_BOOL, DT_COMPLEX64, DT_INT64, DT_UINT64]
device='TPU_SYSTEM'
device='GPU'; T in [DT_HALF]
device='GPU'; T in [DT_BFLOAT16]
device='GPU'; T in [DT_FLOAT]
device='GPU'; T in [DT_DOUBLE]
device='GPU'; T in [DT_INT64]
device='GPU'; T in [DT_UINT16]
device='GPU'; T in [DT_INT16]
device='GPU'; T in [DT_UINT8]
device='GPU'; T in [DT_INT8]
device='GPU'; T in [DT_COMPLEX64]
device='GPU'; T in [DT_COMPLEX128]
device='GPU'; T in [DT_VARIANT]
device='DEFAULT'; T in [DT_STRING]
device='DEFAULT'; T in [DT_VARIANT]
device='DEFAULT'; T in [DT_RESOURCE]
device='CPU'
 [[Identity]] [Op:MakeIterator]

I am able to replicate the issue, please find the gist here.

Having same issue here, although the model seems to work when using the GPU. I believe it may be related to the override of the Keras ImageGenerator but I have not enough knowledge to be sure.

I have the same issue here. For training in TPU, do we have to use the tfrecord or we could use our customized data generator?

The error message indicates you're using uint8, which isn't supported on TPU. You can cast to uint32.

The error message indicates you're using uint8, which isn't supported on TPU. You can cast to uint32.

Could you show how to do that when using a kera.utils.Sequence? Should you override it with a class?

Jhseu, for me it is working now, per your suggestion.
I am converting the uint8 image resulted from tf.io.decode_jpeg to float32 using:
image = tf.image.decode_jpeg("image.jpg", channels=3)
image = tf.cast(image, tf.float32)
If I want to also cast all values between 0 and 1 I use:
image = tf.cast(image, tf.float32)/255.0    # convert image to floats in [0, 1] range
Also, I found using TFRecords to be very important, using just tf.data.Dataset is impossibly slow.
Thank you. I will close the issue.

I have the same issue here. For training in TPU, do we have to use the tfrecord or we could use our customized data generator?

We can use both data but the data should be in cloud storage for tpu training . Also for visualizing the logs to Tensorboard we should train the model only in Cloud TPU . I tried to run the model running in  Colab TPU having the storage bucket but it won't work like that .


Native method not found: com.facebook.crypto.cipher.NativeGCMCipher.nativeEncryptInit

I have run into this error many times in the production, not once in development.
Caused by: java.lang.UnsatisfiedLinkError: Native method not found: com.facebook.crypto.cipher.NativeGCMCipher.nativeEncryptInit

My Proguard rules contains
-keepclasseswithmembernames class * {
    native <methods>;
}

So I wonder why this error still taking place, mostly on 4.4 devices, any help?

This is definitely related to Conceal, I'll check if they have anything with the new version.

I met another issue like this.
Caused by: No implementation found for native Lcom/facebook/crypto/cipher/NativeGCMCipher;.nativeDecryptInit:([B[B)I
java.lang.UnsatisfiedLinkError: Native method not found: com.facebook.crypto.cipher.NativeGCMCipher.nativeDecryptInit:([B[B)I
On 4.4 devices
Could you help me?

java.lang.UnsatisfiedLinkError: No implementation found for int com.facebook.crypto.cipher.NativeGCMCipher.d(byte[], byte[]) (tried Java_com_facebook_crypto_cipher_NativeGCMCipher_d and Java_com_facebook_crypto_cipher_NativeGCMCipher_d___3B_3B)
Same problem, android 7.1.1 and Hawk in version 2.0.1

com.facebook.crypto.cipher.NativeGCMCipher.nativeDecryptInit
NativeGCMCipher.java, line -2
Same Problem  , do not konw how to solve it

Im also facing this problem
 java.lang.UnsatisfiedLinkError: No implementation found for int com.facebook.crypto.cipher.NativeGCMCipher.n(byte[], byte[]) (tried Java_com_facebook_crypto_cipher_NativeGCMCipher_n and Java_com_facebook_crypto_cipher_NativeGCMCipher_n___3B_3B)