compile c++ failed of the version release/0.9

Hello,
I met an error when compile the torchvision of c++ in linux.
The enviroment as belows:
cmake version 3.14.5
torch version 1.8.1+cpu
the command i excute:

git clone https://github.com/pytorch/vision.git
cd vision
git checkout release/0.9
mkdir build && cd build
make -j4
sudo make install

the error log as belows:

-- The C compiler identification is GNU 7.5.0
-- The CXX compiler identification is GNU 7.5.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found Python3: /environment/python/versions/miniconda3-4.7.12/lib/libpython3.7m.so (found version "3.7.4") found components:  Development
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found Torch: /environment/python/versions/miniconda3-4.7.12/lib/python3.7/site-packages/torch/lib/libtorch.so
-- Found ZLIB: /usr/lib/x86_64-linux-gnu/libz.so (found version "1.2.11")
-- Found PNG: /usr/lib/x86_64-linux-gnu/libpng.so (found version "1.6.34")
-- Found JPEG: /usr/lib/x86_64-linux-gnu/libjpeg.so (found version "80")
-- Configuring done
CMake Warning at CMakeLists.txt:73 (add_library):
Cannot generate a safe runtime search path for target torchvision because
files in some directories may conflict with libraries in implicit
directories:
runtime library [libpng16.so.16] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
  /environment/python/versions/miniconda3-4.7.12/lib

Some of these libraries may not be found correctly.

-- Generating done
-- Build files have been written to: /home/featurize/work/yolov5-rt-stack/deployment/vision/build
make: Warning: File 'Makefile' has modification time 79 s in the future
make[1]: Warning: File 'CMakeFiles/Makefile2' has modification time 79 s in the future
make[2]: Warning: File 'CMakeFiles/torchvision.dir/flags.make' has modification time 79 s in the future
Scanning dependencies of target torchvision
make[2]: warning:  Clock skew detected.  Your build may be incomplete.
make[2]: Warning: File 'CMakeFiles/torchvision.dir/flags.make' has modification time 78 s in the future
[  2%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/common_jpeg.cpp.o
[ 10%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_image.cpp.o
[ 10%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_jpeg.cpp.o
[ 10%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_png.cpp.o
[ 13%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/encode_jpeg.cpp.o
c++: internal compiler error: Killed (program cc1plus)

Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
CMakeFiles/torchvision.dir/build.make:101: recipe for target 'CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_png.cpp.o' failed
make[2]: *** [CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_png.cpp.o] Error 4
make[2]: *** Waiting for unfinished jobs....
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
CMakeFiles/torchvision.dir/build.make:88: recipe for target 'CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_jpeg.cpp.o' failed
make[2]: *** [CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_jpeg.cpp.o] Error 4
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
CMakeFiles/torchvision.dir/build.make:114: recipe for target 'CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/encode_jpeg.cpp.o' failed
make[2]: *** [CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/encode_jpeg.cpp.o] Error 4
CMakeFiles/Makefile2:72: recipe for target 'CMakeFiles/torchvision.dir/all' failed
make[1]: *** [CMakeFiles/torchvision.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

Hum, you seem to have gotten an internal compiler error, which is very weird.
Can you check that you are not running out of memory when compiling torchvision with -j 4?

hello,  I rerun and excute make  instead of make -j4. also gets error :

(base) ➜  build git:(release/0.9) make
Scanning dependencies of target torchvision
make[2]: Warning: File 'CMakeFiles/torchvision.dir/depend.make' has modification time 79 s in the future
[  2%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/common_jpeg.cpp.o
[  5%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_image.cpp.o
[  8%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_jpeg.cpp.o
[ 10%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/decode_png.cpp.o
[ 13%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/encode_jpeg.cpp.o
[ 16%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/encode_png.cpp.o
[ 18%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/cpu/read_write_file.cpp.o
[ 21%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/io/image/image.cpp.o
[ 24%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/models/alexnet.cpp.o
[ 27%] Building CXX object CMakeFiles/torchvision.dir/torchvision/csrc/models/densenet.cpp.o
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
CMakeFiles/torchvision.dir/build.make:179: recipe for target 'CMakeFiles/torchvision.dir/torchvision/csrc/models/densenet.cpp.o' failed
make[2]: *** [CMakeFiles/torchvision.dir/torchvision/csrc/models/densenet.cpp.o] Error 4
CMakeFiles/Makefile2:72: recipe for target 'CMakeFiles/torchvision.dir/all' failed
make[1]: *** [CMakeFiles/torchvision.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2


Support for older versions?

I confirm that I have ...

[X] ...tried the latest build to make sure my feature doesn't exist already.
[X] ...checked the realism of my feature and i think it can be executed.
[X] ...checked other existing issues for my feature request.

Description
Should I make versions for older Minecraft versions?
It's not a lot of work but it's tedious and I wonder if you guys even want/need that.
So reply in the comments, should I add releases for older versions, and if so, which versions?
Code Example
No response

The 1.17 version would be nice :)

The 1.17 version would be nice :)

Consider it done :)
Will release tomorrow

The 1.17 version would be nice :)

https://github.com/CraZyRc/WhackaMole/releases/tag/1.17.1
Here you go, 1.17 will be taken in account with updates so check out the 1.17 tag for updates :)


setFeedURL documentation does not match TypeScript types

According to https://www.electronjs.org/docs/tutorial/updates#implementing-updates-in-your-app
I should be able to call autoUpdater.setFeedURL("this is a string").

Hey @NaridaL! Thanks for the report (and sorry for the delay).
See https://github.com/electron/electron/pull/23298 for a fix to this. For future reference, please note that all the website's docs live in the main electron/electron repository, so docs-related issues should be filed there!


ChainAPI should support CSV download

When working with data, many people are comfortable with CSV / xls files but don't know how to query web APIs. Making the data downloadable in CSV would be helpful for them.

It may also be worth looking into HDF5 and/or NetCDF (or others?) as more space-efficient formats for larger datasets.


Changed the colour of font in footer to black to increase visibility

Changed the colour of font in footer to black and decreased the transition of y-translate of social icons to make it look better

Changed the colour of font in footer to black and decreased the transition of y-translate of social icons to make it look better

not required


Fix template rendering for compatibility with Django 1.10+

Update test matrix for Django 1.8, 1.9, 1.10 and master
Fix template deprecations in 1.8, 1.9 and incompatibility in Django 1.10 and up
Note that the template fix is backwards incompatible; any plain text templates must now invoke autoescape off explicitly
Backfill changelog for prior releases

resolves #68

Coverage decreased (-0.04%) to 71.811% when pulling 28dd70c7216d58e20786faa0a816f1dbe860e3ec on fix-template-deprecations into 085f6daf56de9bd3bd905a12d669d41e4754d15b on master.

Coverage decreased (-0.04%) to 71.811% when pulling 28dd70c7216d58e20786faa0a816f1dbe860e3ec on fix-template-deprecations into 085f6daf56de9bd3bd905a12d669d41e4754d15b on master.


Unexpected attribute 'query' found in query criteria

It doesnt like query: (ref) => ref.where('d.count', '==', 1) in the following:
   geoQuery = geoFirestore.query({
      center: new firebase.firestore.GeoPoint(mapCenter.lat(), mapCenter.lng()),
      radius: radius,
      query: (ref) => ref.where('d.count', '==', 1)
    });

Has error: Unexpected attribute 'query' found in query criteria
Any thoughts?

Could we see the package.json? (Or a broken example, like a stackblitz link?)

Closing this for now, but if you get the chance to share a working (broken) example we can definitely reopen this!

See example here it works when below is removed...
query: (ref) => ref.where('d.count', '==', 1)

@jamiegalbreath I finally got to test it, I see that the JSFiddle was using version 2.0.2, however custom queries weren't available until v 2.1.0. Try updating to the latest version 2.2.1 and you should be all set.
You can see that here, however it's not indexed 😅


[IRGen] Restore the old code path for emitting existential type metadata for plain protocol and protocol composition types.

These types should always be wrapped in ExistentialType, but there isn't sufficient validation of this throughout the compiler yet. Change the fatal error when the metadata request sees these plain types to an assert, and restore the old type metadata emission path for protocol and protocol composition types to avoid crashing in those cases.
Resolves: rdar://92413116

@swift-ci please smoke test


Drag And Drop Causing Crash on X11

When testing Drag and drop with the RGFW_dnd event, I successfully got the files, but the program would crash immediately in the next RGFW_windowcheckEvent call.
My DND code is just this
case RGFW_dnd:
    printf("Got %d files\n", win->event.droppedFilesCount);
    for (u16 i = 0 ; i < win->event.droppedFilesCount ; i++) {
        printf("File %d: %s\n", i, win->event.droppedFiles[i]);
    }
    break;

Xorg logs this error:
X Error of failed request:  BadValue (integer parameter out of range for operation)
  Major opcode of failed request:  25 (X_SendEvent)
  Value in failed request:  0x0
  Serial number of failed request:  2825
  Current serial number in output stream:  2825

After looking into it I noticed something weird with the code from here: (https://github.com/ColleagueRiley/X11-Drag-n-Drop).

the drop has ended and XDnD versions 2 and older require the target to tell the source when the drop has ended

if (version >= 2) {
    reply.xclient.message_type = XdndFinished;
    reply.xclient.data.l[1] = result;
    reply.xclient.data.l[2] = XdndActionCopy;

    XSendEvent((Display*) display, source, False, NoEventMask, &reply);
    XFlush((Display*) display);
}

Versions 2 and older? But the condition works in the opposite way.
Switching the condition at the end of the case SelectionNotify: part of the eventHandler to:
if (xdnd.version <= 2)  (about line 3160 in my header version) fixes the problem for me, and I can get the files and continue on.

@op4946 I believe this is a typo in the article rather than the code.
GLFW's code looks like this:
                if (_glfw.x11.xdnd.version >= 2)
                {
                    XEvent reply = { ClientMessage };
                    reply.xclient.window = _glfw.x11.xdnd.source;
                    reply.xclient.message_type = _glfw.x11.XdndFinished;
                    reply.xclient.format = 32;
                    reply.xclient.data.l[0] = window->x11.handle;
                    reply.xclient.data.l[1] = result;
                    reply.xclient.data.l[2] = _glfw.x11.XdndActionCopy;

                    XSendEvent(_glfw.x11.display, _glfw.x11.xdnd.source,
                               False, NoEventMask, &reply);
                    XFlush(_glfw.x11.display);
                }

source (https://github.com/glfw/glfw/blob/21fea01161e0d6b70c0c5c1f52dc8e7a7df14a50/src/x11_window.c#L1720)
I'm not entirely sure why it has this error for you. I think there is a problem somewhere else in the code.
Do you know if the X11 example code from the tutorial works on your system? I don't have that error, but the example can help to find where the issue is.
Thanks a lot for filing this issue!

@op4946 Just a quick update, I just tested the basic/basic example and I get the same error. I wasn't getting this error before so it must be caused by something that changed now the last time I tested it.

@op4946 This should be finished now with this commit. (https://github.com/ColleagueRiley/RGFW/commit/ef00e8fde3c73f1aeb3fa355370b8e5d73f671b3)
The reason why changing the logic worked is because it didn't run this part of the code. However, that would lead to some issues with the X11 client because the XdndFinish reply is not being sent.
I should also fix the wording in the tutorial, thanks for pointing this out!

I tested your fix on my computer, and everything now works. With a new clone of the repository, the examples like basic and callbacks all work fine (the examples had the same error before), and my code from before works as well.


On iOS 8 in landscape mode, if the navigation bar opens, game.height doesn't change

Does Phaser can manage this under the hood and I missed something or I've to do it myself?
Opening the iOS navigation bar dispatch setResizeCallback, then I will have to setGameSize accordingly?

I tried to manage it on my side, but since I use ScaleManager.RESIZE, setGameSizedoesn't have any effect.
So Phaser should definitely handle it on this own.

If the State.resize function gets called (or the onSizeChange Signal gets dispatched) then you can adapt your game size accordingly. If neither of those two things happen then iOS isn't triggering a window.resize event, so there's nothing you can actually do directly from Phaser, as it doesn't consider the window as having changed size.


Would love a flag to prevent it setting the background

Sometimes I'd just like to use it for the --output.
I'd be happy to submit a PR if you'd like.

Hi. Yeah, @lgastako, happy to accept a PR!


Duplicate comments getting generated while syncing extensions from Axis

Describe the bug in current situation
While syncing extensions from axis, duplicate comments are getting generated.
Link bug to the User Story
Impact of this bug
Describe the impact, i.e. what the impact is, and number of users impacted.
Chance of Occurring (high/medium/low/very low)
High
Pre Conditions: which Env, any pre-requesites or assumptions to execute steps?
Steps to Reproduce
Steps to reproduce the behavior:

Go to any request with extensions in axis.
Sync up the request.
Comments will be generated for the synced up extensions.
Add 1 more extension in axis and sync again.
5.Go to comments tab and See
OR
Go to any request with extensions in axis.
Sync up the request.
Comments will be generated for the synced up extensions.
Delete 1 of the synced extensions from FOI Request.
Again sync up the extensions.
6.Go to comments tab and See

Actual/ observed behaviour/ results
Duplicated extension comments will be generated for already synced extensions.
Expected behaviour
Whenever extensions are synced up, comments should be generated only for the newly synced up axis extensions and not for the already synced extensions.
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.![screengrab.png]

From now on extension comments will only be generated for each of the extensions being synced & no duplicates. Also if any extension are added manually in FOI system and then extensions are synced over from axis, comments will be generated for the extensions that gets deleted as well.

Thanks, @aparna-aot


Info sign and severity not matching with the level of the log

Environment details

OS: Ubuntu 18.04
Node.js version: 8.11.3
npm version: 6.4.1
@google-cloud/logging-bunyan version: 1.8.12

Steps to reproduce

Implement the code as per the doc says :

const loggingBunyan = new LoggingBunyan({
  projectId: projectId,
  keyFilename: 'keyfilenamePath.json'
});
logger = bunyan.createLogger({
        name: process.env.STACKDRIVER_CONTAINER_NAME,
        streams: [
          { stream: process.stdout, level: 'info' },
          loggingBunyan.stream('info')
        ]
});

logger.error('logger error initialized');
logger.info('logger info initialized');

Run the application and check inside Stackdriver GCP :

logger.error will always have the blue info sign but with a level at 50 and a severity at INFO.
So filtering the logs by the level is impossible at the meantime.
Thanks!

Hi @realiceman. I see that you are logging to the console in addition to streaming to the Bunyan Stackdriver stream. Both the console and the Bunyan streams are captured and available in the Stackdriver Logging UI. Make sure you are looking at the correct log destination.
Here's an example (the log name will be dependent on what how you named your Bunyan log, the default is bunyan_log):

Note that the insertId field generated by this library has this unique pattern starting with ....
Can you make sure you are looking at the correct log stream in the UI?

Hi @ofrobots , yes it is the correct destination. But unfortunately the blue info sign and severity are not matching the log level.
Example : log.error("test");  , still blue info sign and severity marked at INFO level, but the key "level" is at 50.
thank you

@realiceman This is what I see running your code:

Looking at the insertId in your log screenshot, the insert id does not match the expected pattern for the logs generated by the @google-cloud/logging-bunyan stream. I still suspect that you are looking at the incorrect Log in the UI.
What environment are you deploying your application to? App Engine? GKE? GCE? For Kubernetes, you will have to make sure to select 'GKE container' in the first drop down, and then select 'bunyan_log' in the second drop down. If you cannot find these log destinations, can you provide screenshots of what you see in the first and second drop-down lists?

Actually i followed the tutorial inside the npm page https://www.npmjs.com/package/@google-cloud/logging-bunyan and the sample on github . i've done the test again without "{stream: process.stdout}" and still i got the same issue.
Is there any chance you could publish your project "samples_express_applog" ?

@realiceman The samples_express_applog screenshot above was from the express sample in this repo deployed to App Engine standard.
Can you let me know where you are running your application? Is it running on Kubernetes, App Engine, Compute Engine, locally, or elsewhere?

@ofrobots  Here is my scenario :
the nodejs app is a docker image, deployed in GCP Kubernetes. So the logging works because it logs things, but not as well as it supposed to be like your sample app.

@realiceman Here's a screenshot bunyan app on GKE that I created using the sample you're following.

Please note the log name I have selected in the drop-downs. I am restricting only to the bunyan_log.
If you switch the view to All logs you will start seeing the automatically captured stdout and stderr in addition to the @google-cloud/logging-bunyan log stream. Note that the automatically captured stdout will not have the log levels recognized.

I've uploaded all the source code for the sample into this git repository (https://github.com/ofrobots/gke-bunyan-test). You should be able to deploy this into your own project to see this working.
Since everything is working correctly as far as I can still, I am going to close this issue. If you think there is a bug in the code or things are not working correctly, please add a comment and I can reopen the issue.


EXIF longitudinal and latitudinal Image based location detection

Checklist

[X] I've searched through the plugins to make sure this feature isn't already available, or think it shouldn't require a plugin.
[X] I've searched through the current issues to make sure this feature hasn't been requested already.
[X] I agree to follow the Code of Conduct that this project adheres to.

Motivation
The benefit of this feature is that anyone can be EXIF (Exchangeable image file format) is a standard format used for storing metadata in image files taken by digital cameras and other devices. This metadata can include information such as the date and time the photo was taken, camera settings, and also GPS coordinates of the location where the photo was taken.
Extracting the GPS coordinates from EXIF data can be useful for several reasons. For example:
Location tagging: With GPS coordinates, you can easily tag your photos with the location where they were taken. This can be useful for organizing and categorizing your photos, as well as for sharing them on social media platforms that support location-based tagging.
Geotagging: GPS coordinates can be used to geotag photos on maps, enabling users to see where the photo was taken in relation to other locations and landmarks.
Photo management: GPS coordinates can be used to sort and organize photos based on location, making it easier to find and manage large collections of photos.
Memory keeping: GPS coordinates can help you remember the location where you took a photo, which can be useful for documenting your travels and other important events.
Suggested solution
To extract GPS coordinates from EXIF data, you can use a combination of programming languages and libraries. Here is one possible stack you can use:
Python: Python is a popular programming language for data processing and analysis. It has many libraries for working with image files and extracting EXIF metadata.
Pillow: Pillow is a Python imaging library that provides support for opening and manipulating image files. It also includes support for reading and writing EXIF metadata.
ExifRead: ExifRead is a Python library for extracting EXIF metadata from image files. It provides a simple interface for accessing EXIF data, including GPS coordinates.
Mapbox: Mapbox is a popular mapping platform that provides tools for displaying maps and visualizing geospatial data. It also includes support for geocoding and reverse geocoding, which can be used to convert GPS coordinates into human-readable locations.
Alternatives considered
No response

I'm closing this just on a hunch: it reads like ChatGPT. The "motivation" is rather "explain why exif can be useful".
Otherwise, displaying photos based on exif metadata is best left to a plugin, perhaps leveraging https://www.npmjs.com/package/exif-parser and the File API.

Agreed. This sound like something that could be a Leaflet plugin, not something that belongs in Leaflet core.


Support for variables?

Does sloth support variables that can be used in queries? Or are there any plans to add support for variables? Might be very useful and should be easy to implement with sloths "compile the real query" approach.

You could accomplish that using SLI plugins


Adding options to skip/ignore remote config when extending

resolves #180
When loading in user configs we don't always want to import remote non-trusted code in code-based config files (.js, .ts, ...). This PR adds the option to disable remote extending while still supporting extensions of local config files.

@pi0 Hey, I don't want to annoy you if you're busy. I would be really happy if you could give a quick feedback on this and if or how it could be merged. There is also another PR open for a while, just a quick comment would let us know that it has been seen and if it has a chance of being merged. This is a really valuable package and thank you for maintaining it with many other packages 🫡.


Configure foreground match count in Find/Replace widget

VS Code version: 1.35.1
Your operating system: Mac OS 10.14.6 Beta (18G48f)
List of extensions that you have installed: Advanced New File, C/C++, Color Highlight, Python, SpellChecker, Studio Icons, Word Count
Reproducible steps (1... 2... 3...) that cause the issue:

Create new file
Open settings.json and set "editor.foreground" to some high-contrast color. I'll use #3daef3 as an example
Enter some string of text into the editor. I'll use "code" for example
Open Find/Replace
Type "code" into the Find/Replace search box
Observe that foreground of the "1 of 1" text to the right of the search box in Find/Replace uses #3daef3 as foreground

What you expected to see, versus what you actually saw: I expected to find a setting that would let me change the foreground color of the "1 of 1" text — or any other permutation of search results/match counts in a file other than "No Results" — to something other than the editor.foreground color. But I could not find anything along those lines. I checked to see if there was an editorWidget.foreground setting just as there is an editorWidget.background setting. But there does not seem to be such an option. It would be nice to be able to customize this element of the Find/Replace widget in order to further enhance color themes.
Images, animations, or a link to a video showing the issue occurring:

A code snippet that demonstrates the issue or a link to a code repository the developers can easily pull down to recreate the issue locally
"editor.foreground": "#3daef3",
Thanks!

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Not possible to explicitly change the foreground text color for widgets (#76274)

Thanks for flagging that issue, vscodebot! And thank you @mkanet -- yours and mine are very similar but slightly different. That one points to instances of "No Results," whereas mine points to instances in which there are matches to text entered in the search box within the Find/Replace widget. However, we both propose the same solution, which is to expose an editorWidget.foreground setting option.

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:
Not possible to explicitly change the foreground text color for widgets (#76274)
Thanks for flagging that issue, vscodebot! And thank you @mkanet -- yours and mine are very similar but slightly different. That one points to instances of "No Results," whereas mine points to instances in which there are matches to text entered in the search box within the Find/Replace widget. However, we both propose the same solution, which is to expose an editorWidget.foreground setting option.

@jx22 It's good to know that I'm not the only person to notice this.  Hopefully, we'll see an editorWidget.foreground setting option in the not-so-distant future.

I will call this a dupe of https://github.com/microsoft/vscode/issues/76274 because I think that any solution to that one would cover this as well.


Manage Helm chart repo configuration within cluster

Motivation
Currently, Helm chart repository configurations are kept in local .config/helm/repositories.yaml file only and its content it managed by issuing helm repo add/remove commands. Such approach imposes the following issues:

Right configuration should be distributed and kept in sync on all relevant machines
Repository configuration cannot be added on a read-only filesystem, forcing the usage of absolute chart URLs
Inability to have repository configurations per a cluster
Inability to share repository configurations in a team

Proposal
The HelmChartRepository is a simple cluster-scoped CR for letting cluster admins define Helm chart repository configurations in a cluster.
apiVersion: config.helm.sh/v1alpha1
kind: HelmChartRepository
metadata:
  name: myCharts
spec:
  url: http://my.chart-repo.org/stable

  # optional and only needed for UI purposes
  displayName: myChartRepo

  # optional and only needed for UI purposes
  description: my private chart repo

  # set to true if need to be disabled temporarly
  disabled: true

  # other fields from
  # https://github.com/helm/helm/blob/master/pkg/repo/chartrepo.go#L42

The proposed CR should be semantically equivalent to repo.Entry structure.
This change would enable Helm to read repository configurations from the configured cluster as well and use them in appropriate actions.
Configuring Repositories
Cluster admin can add a number chart repositories:
$ cat <<EOF | kubectl apply -f -
apiVersion: config.helm.sh/v1alpha1
kind: HelmChartRepository
metadata:
  name: stable
spec:
  url: https://kubernetes-charts.storage.googleapis.com

  displayName: Public Helm stable charts

---
apiVersion: config.helm.sh/v1alpha1
kind: HelmChartRepository
metadata:
  name: myCharts
spec:
  url: http://my.chart-repo.org/stable

  displayName: my private chart repo
EOF

Usage
The cluster treats them as any other resource:
$ kubectl get helmchartrepositories
NAME          AGE
stable        1m
myCharts      1m

and Helm can access them and use them as they were added using helm repo add command:
# repositories configured in the cluster.
$ helm repo list
NAME     	URL
stable   	https://kubernetes-charts.storage.googleapis.com
myCharts	https://kubernetes-charts-incubator.storage.googleapis.com

# install a chart from the "stable" repo defined in the cluster
$ helm install mysql stable/mysql

NOTE: Storing repository configuration locally still works and is perform as usual using helm repo add/remove commands. However, the configuration stored in the cluster is managed using kubectl/K8S API.
Benefits

Easy configuration sharing across teams/infrastructure
Simpler team onboarding
Repository configuration can be cluster-scoped, e.g. stable repo in cluster A must not be the same as stable repo in cluster B
Helm Hub can publish curated configurations, easily consumable by cluster admins
Cluster admins could white list repositories
A step forward to running Helm in read-only environments

Open Questions

Should we allow namespaced HelmChartRepository?

cc @technosophos

I like the proposal, it for sure will make easier to use helm CI/CD pipelines

I want to add some additional context that needs to be considered... and I have some questions based on the context...
First, Helm serves different user profiles/roles. The highest one in priority order is the application operator. We've worked to make this possible without requiring anything from the cluster operator.
Helm can be used in a wide variety of clusters. That includes multi-tenant clusters where tenants are unaware of each other and often the cluster operator does not have contextual knowledge of what the application operators are doing.
This proposal prioritizes the cluster operator. This makes me wonder, how do two different teams in a multi-tenant cluster work in this setup? I get the impression that this proposal makes assumptions about how the Kubernetes cluster is setup and used.
Could we get the need written in the form of a use case? I would like to better understand the situation. Is this for a cluster operator to provide repos? How does the cluster operator know the right repos for all the tenants of the cluster? Is this for teams to share repos? Teams sharing is different than cluster operators because the role of a cluster user is different from that of a cluster operator. I'm curious of the context here.
Second, there are cases where application operators cannot install CRDs and the cluster operators will not install them. Helm attempts to (and does currently) work in this situation. This is why Helm v3 uses secrets for storage instead of CRs. We have Helm users that take advantage of this today.
How would this work in that situation? Would it be an inaccessible feature?
Third, when using the --dry-run flag or using helm template Helm assumes no cluster connection is possible. There are security implications around this, as well, given the way these are sometimes used. How would a feature like this work in the situation where someone performs one of these functions?

@rimusz what about CI/CD pipelines will this make easier?
I ask because right now you can install a chart without adding a repository by specifying a URL. If you use chartmuseum and push you can push to a URL instead of a specified repository. There are many things that can be done today where no repository is required to be setup.
What specific things need improving? I ask because I honestly would like to either work on them or document the things people aren't aware of that may already exist.

cc @jnpacker

HI @mattfarina ,
Disclosure: I'm co-sponsoring this proposal with @waveywaves , @pedjak & @jnpacker .

First, Helm serves different user profiles/roles. The highest one in priority order is the application operator. We've worked to make this possible without requiring anything from the cluster operator.

Agreed!

This proposal prioritizes the cluster operator. This makes me wonder, how do two different teams in a multi-tenant cluster work in this setup? I get the impression that this proposal makes assumptions about how the Kubernetes cluster is setup and used.

How does the cluster operator know the right repos for all the tenants of the cluster?

I wanted to validate my understanding of the cluster operator profile - so here it goes :)

A cluster operator stands up and operates a Kubernetes cluster. This includes elements such as the control plane, nodes, and elements in the stack below these

Even though HelmChartRepositories tend to define what "applications" becomes available to users - and that could impact the infra setup decisions, I would still consider this to be primarily a cluster user task even though lines could be blurred.

I would like to better understand the situation. Is this for a cluster operator to provide repos?

While I can think of cases where a cluster operator might want to add HelmChartRepositories  but I do think this activity might make sense for cluster users.
As an example, I do see myself setting up clusters and then passing it on to another 'privileged' team member to configure which chart repositories are to be made available on the cluster based on the business unit she works on.

Is this for teams to share repos? Teams sharing is different than cluster operators because the role of a cluster user is different from that of a cluster operator. I'm curious of the context here.

Teams and individuals.
Example,
I, as a cluster user, would like to make specific  HelmChartRepository resourceNames view-able to specific users/groups, and that should drive the output of helm repo list.
Or, I, as a cluster user, would like to make them available for system:authenticated

( addressing the rest of the questions too, shortly )

How would this work in that situation? Would it be an inaccessible feature?
Third, when using the --dry-run flag or using helm template Helm assumes no cluster connection is possible.

we can think of following ways to approach this.
with cluster connection.
Basic requirement for dry-run and template is Chart itself, currently charts are extracted using  index.yaml/repositories.yaml from local config.
we can use an extra flag (something like --remote)  in dry-run or in template to get the chart listing by connecting to cluster using k8s api.
without cluster connection.
This feature won't be available if repos are configured in cluster with CRD, If they are configured in local --dry-run and template should work off the shelf.
cc: @mattfarina  @sbose78

@mattfarina

This proposal prioritizes the cluster operator. This makes me wonder, how do two different teams in a multi-tenant cluster work in this setup? I get the impression that this proposal makes assumptions about how the Kubernetes cluster is setup and used.

Instances of the proposed CR are optional. If found/accessible by Helm, they can be consumed. Otherwise, business as usual.
The proposal focus is on enabling cluster operator/teams to share repository configuration with users/team members, and keeping it in sync, without a need to instruct users how to manage their local setups, via a number of helm repo add commands.
The configuration can be global (cluster-scoped CR). but as stated in the open questions above, it could be namespaced as well, giving team a possibility to share configuration without a need to involve the cluster operator.

Third, when using the --dry-run flag or using helm template Helm assumes no cluster connection is possible. There are security implications around this, as well, given the way these are sometimes used. How would a feature like this work in the situation where someone performs one of these functions?

The help message for --dry-run says simulate operation XXX. It does not state that talking to the cluster is not allowed. However, if we want to stick to that, clearly, repository configurations stored in the cluster cannot be accessed and this might impact the operation under dry-run. In case of helm install/template/update we could give a hint to users that repository under given name cannot be found locally and we are not allowed to check if the request repo configuration is available in the cluster.

@mattfarina can you take a look on my last comment in the thread and let me know if your concerns are resolved now?

@mattfarina do you still have concerns? cc: @pedjak @sbose78

I see there is a new process! We'll write a HIP with more details and clarification, post the link to the new HIP back here then this can be closed out.


Train.py Error

I tried to run both train.py and recognize,py but I have the same error like this. Noted that i already place the data or the pre trained model at the right folder. By reading the error, my python might be the problem, please help me to fix it !
/usr/lib/pymodules/python2.7/matplotlib/rcsetup.py:378: UserWarning: tk.pythoninspect is obsolete, and has no effect
warnings.warn("tk.pythoninspect is obsolete, and has no effect")
Traceback (most recent call last):
File "train.py", line 3, in
from data import VCTK
File "/home/phong/speech-to-text-wavenet-master/data.py", line 5, in
import librosa
File "/usr/local/lib/python2.7/dist-packages/librosa/init.py", line 18, in
from . import display
File "/usr/local/lib/python2.7/dist-packages/librosa/display.py", line 30, in
_matplotlibrc = copy.deepcopy(mpl.rcParams)
File "/usr/lib/python2.7/copy.py", line 190, in deepcopy
y = _reconstruct(x, rv, 1, memo)
File "/usr/lib/python2.7/copy.py", line 358, in _reconstruct
y[key] = value
File "/usr/lib/pymodules/python2.7/matplotlib/init.py", line 808, in setitem
cval = self.validatekey
File "/usr/lib/pymodules/python2.7/matplotlib/rcsetup.py", line 95, in validate_bool_maybe_none
raise ValueError('Could not convert "%s" to boolean' % b)
ValueError: Could not convert "None" to boolean

python -c "import tensorflow; print tensorflow.version"
should be 0.11.0
Just got working with this update.
On my Mac
`export TF_BINARY_URL=https://storage.googleapis.com/tensorflow/mac/cpu/tensorflow-0.11.0-py2-none-any.whl
sudo pip install --upgrade $TF_BINARY_URL'

`I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcurand.so locally
INFO:tensorflow:1129:11:51:04.747:data.py:60] VCTK corpus loaded.(total data=36395, total batch=2274)
Traceback (most recent call last):
File "train.py", line 70, in
z, s = res_block(z, size=7, rate=r)
File "train.py", line 52, in res_block
conv_gate = tensor.sg_aconv1d(size=size, rate=rate,  act='sigmoid', bn=True)
File "/usr/local/lib/python2.7/dist-packages/sugartensor/sg_main.py", line 149, in wrapper
out = func(tensor, opt)
File "/usr/local/lib/python2.7/dist-packages/sugartensor/sg_layer.py", line 91, in sg_aconv1d
w = init.he_uniform('W', (1, opt.size, opt.in_dim, opt.dim))
File "/usr/local/lib/python2.7/dist-packages/sugartensor/sg_initializer.py", line 31, in he_uniform
return uniform(name, shape, s, dtype)
File "/usr/local/lib/python2.7/dist-packages/sugartensor/sg_initializer.py", line 20, in uniform
initializer=tf.random_uniform_initializer(minval=-scale, maxval=scale))
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 1024, in get_variable
custom_getter=custom_getter)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 850, in get_variable
custom_getter=custom_getter)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 346, in get_variable
validate_shape=validate_shape)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 331, in _true_getter
caching_device=caching_device, validate_shape=validate_shape)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 632, in _get_single_variable
name, "".join(traceback.format_list(tb))))
ValueError: Variable layers/aconv1d_1/W already exists, disallowed. Did you mean to set reuse=True in VarScope? Originally defined at:
File "/usr/local/lib/python2.7/dist-packages/sugartensor/sg_initializer.py", line 20, in uniform
initializer=tf.random_uniform_initializer(minval=-scale, maxval=scale))
File "/usr/local/lib/python2.7/dist-packages/sugartensor/sg_initializer.py", line 31, in he_uniform
return uniform(name, shape, s, dtype)
File "/usr/local/lib/python2.7/dist-packages/sugartensor/sg_layer.py", line 91, in sg_aconv1d
w = init.he_uniform('W', (1, opt.size, opt.in_dim, opt.dim))
`
I installed both tf 0.11 & 0.12 and still got this problem.

You can check your tensorflow version using python 3 with python -c "import tensorflow as tf; print(tf.__version__)"


PhamVanBach_2252057_Assignment241_2DCutiingStockProblem

My all code files

Please revise the PR in Dec 16. Otherwise, your group will get 0 score for code.

I just add my code files to the submission in Lms but it seems like the web
can not load. Can you delay it until December 17
Vào Th 2, 16 thg 12, 2024 lúc 20:01 Duc Quang Nguyen <
@.***> đã viết:

Please revise the PR in Dec 16. Otherwise, your group will get 0 score for
code.
—
Reply to this email directly, view it on GitHub
https://github.com/martinakaduc/MM241-Assignment/pull/350#issuecomment-2545575543,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BIIEHOLOR6MN5ZUFUEUH2OL2F3FLHAVCNFSM6AAAAABTUWYFAOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBVGU3TKNJUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update Templates When Moved/Renamed

Is your feature request related to a problem? Please describe.
When you move/re-name a template it doesn't auto-update so you need to go back and change it.
Describe the solution you'd like
Auto-update.

So the request is to update the selected template associated with new rows?
It requires a permanent observer for every ddbb you have in all your vault, I think its an overkill to reduce 1click

Maybe we can add a notice to show that the template not exist anymore

Essentially I know Obsidian has the functionality so where you move/rename something it updates where it is referenced everywhere else. Was hoping the same could be done for the templates

I can not use that native architecture in this case because the template is one option more of the ddbb yaml, not a link. I am not sure if we can save that property as an internal link to use that functionality. The problem in that case will be to read it

I am going to discard ( at the moment) this issue. Its not possible with the actual config


fix(lazy): include default locale in the main bundle

This avoid extra request for the locale that is always needed anyway.
Resolves #818

Dup


Version updates for Azure Identity release

API change check
API changes are not detected in this pull request.


Detect disappeared workers.

Fixes https://github.com/RelationalAI-oss/XUnit.jl/issues/4

In addition, it would be nice to only mark the one specific test as failed and restart the worked, but I'm not familiar enough with the state that gets set-up on the worker (it sees fairly intricate) to quickly implement this.

it would be nice to only mark the one specific test as failed and restart the worked

@maleadt I'll try to add this in a separate PR.


[BUG] Localized name not displaying correctly

For the scripts that newly take advantage of localization options, returning t.menu, t.menu, t.desc doesn't seem to have the desired effect. In my Lua menu, I'm seeing the name of the script file (e.g., baseline_move_reset.lua) instead of the expected text.
Win 11, Fin 27.4.1, RGPLua 0.70. For some reason, I seem to be unable to take a screenshot inside of Finale at the moment. I'm using finale_lua_menus.txt to organize things under Plug-ins | Lua, but I see the same behavior when I delete that file.
cc: @rpatters1

Do any of the additional menu items appear?

OK, I have reproduced it. Will explore later today.

This issue is that RGP Lua pre-0.71 is search for this line of code:
function plugindef()

I have fixed this issue by modifying the localized plugindef function definitions as follows:
function plugindef(locale)
--[[
-- This comment allows RGP Lua pre-0.71 to find the plugindef function
function plugindef()
--]]

As you can see, this allows any version of RGP Lua to find a valid compilable Lua function. (It also works on JW Lua, which appears only to search for function plugindef, like 0.71 does now.)
The only reason I am not sure it will work is I need to be sure the bundler does no strip out these comments.

Maybe we could skip the stripping of comments for the script itself, and only do it for libraries. Or maybe we could actually inject this comment into each plugindef function.

I think it's likely that the bundler will strip that comment, but I'm not at my computer. And injecting it would solve the issue for dist scripts but not src.

The bundler does strip that comment, but it's easy to tell it not to strip comments from plugindef(). Is that the approach you'd like to take?
In terms of aesthetics, would JW/RGPLua be happy with just a single-line comment, or does it need to be formatted just that way? Does function need to be flush left?

It has to be a multiline comment, but the function statement could be preceded by whitespace, e.g.
--[[
   function plugindef()
--]]

I think it looks better with no whitespace, but ymmv.
The reason it has to be a multiline comment is that the parser looking for the function is quite basic. It strips anything after --, but it doesn't know about multiline comments. So it takes the above function plugindef() as the beginning of the function. As luck would have it, --]] is interpreted by Lua either as the terminator of a multiline comment or as a single line comment.

I think not stripping comments out of plugindef makes sense.

Fixed in #694


User provided block device read etc.

Hi. First of all, great work! I really enjoy the web simulator.
To implement the user needs to supply these functions:
.read  = user_provided_block_device_read,
.prog  = user_provided_block_device_prog,
.erase = user_provided_block_device_erase,
.sync  = user_provided_block_device_sync,

Do you have an example of how these should look?
E.g. like the examples here: https://github.com/pellepl/spiffs/wiki/Integrate-spiffs
I'm asking because it seems the function arguments required are at a layer of abstraction higher than a driver, which usually just has: (SPI handle, address, size, *buffer).
Best regards
Frederik

Ok, i found you example in the mbed wrapper. Now I just need to know what data type buffer needs to be? I'm assuming uint8_t due to SPI being 8-bit hardware, but the void type confuses me

Ah, thanks!
The easiest route to using littlefs is to use mbed OS if you can. There the SPI drivers are already taken care of you, so in most cases you just have to choose the right pins (example).
For porting littlefs directly, here's the declaration of the block device functions:
https://github.com/geky/littlefs/blob/5ee20e8d774adf0bb538269870b3552fdfc0e046/lfs.h#L95-L114
For example, read gives you these arguments:

const struct lfs_config *cfg - This is a pointer to the config struct itself. This lets you access properties used by littlefs, such as cfg->block_size or cfg->block_count.
You can also use cfg to get access to a SPI object by storing the object in cfg->context. The context variable is only used to pass context to the block device driver.

lfs_block_t block - This is a 32 bit block number. The size of a block is determined by cfg->block_size.

lfs_off_t off - This is a 32 bit byte offset into the current block. The address can be determined by this formula block*cfg->block_size + off.

const void *buffer - This is the buffer for either storing the data or it contains the data to program. It is a void* to match functions like memcpy, though you can just treat it like a uint8_t*.

lfs_size_t size - This is a 32 bit size that indicates how many bytes to read/program.

Here's a quick example that gets the types you mentioned:
int lfs_bd_read(const struct lfs_config *cfg, lfs_block_t block,
        lfs_off_t off, void *buffer, lfs_size_t size) {

    SPI *spi = cfg->context; // must have been assigned before lfs_mount was called
    uint32_t spi_address = block*cfg->block_size + off;
    uint32_t spi_size = size;
    uint8_t *spi_data = buffer;

    blahblahblah

    return 0;
}

You can also look into the existing ports as examples in case it helps:

the emulated block device for testing littlefs
the mbed block device wrapper
the littlefs-fuse wrapper

Hopefully that helps


[Request for external] Codeunit 740 VAT Report Mediator

Please make function GetVATReportConfiguration in the Codeunit 740 in the W1 Version externally available.

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


Windows Installation

❓ How to do something using VISSL
I'm trying to install VISSL on my windows computer that has an NVIDIA GPU. I'm using the command:
' conda install -c vissl -c iopath -c conda-forge -c pytorch -c defaults apex vissl '
but I receive a not found package for apex and VISSL

Hi @marcomameli1992 , thank you for reaching out. We don't provide conda packages for Windows. However, you can install VISSL from source following our instructions. For apex, you should check the apex github repo if it works for windows . If it doesn't, then I think you can just skip apex. For most of the apex functionality, pytorch provides everything now and e can help you with that.

Closing this because we haven't heard back in awhile. @marcomameli1992 let us know if you want us to re-open/need any additional guidance!


Is there anyway to export vertex coordinates in gdspy

Hi,
I am looking for a way to export the vertex coordinates of gds files in python. I believe gdspy would be a good candidate to do it. Could you let me know how to export those values from gdspy. Or I would like to request to update that function unless the current version is possible.
Thanks,
Sooyong

@phylove01 This is something that can be done more or less easily. You can load the library and use the get_polygons method on the top cell of the layout (or iterate over them, if there is more than 1 top cell).
This should give you a good starting point:
import gdspy

lib = gdspy.GdsLibrary(infile="filename.gds")
for cell in lib.top_level():
    for coordinates in cell.get_polygons():
        # numpy array with dimension (N, 2) with all coordinates
        print("Coordinates:", coordinates)

Thank you for your help! It was really helpful to develop a personal tool. I have one more question about your lib. I am gonna leave it on the other issue page. Thank you again!


Add Client Credentials Grant flow

I'd like add Client Credentials Grant flow
STANDARD
https://tools.ietf.org/html/rfc6749#section-4.4
An implementation
https://github.com/googlesamples/apps-script-oauth2/issues/81#issuecomment-334319455

Twitter's app-only authentication also requires grant_type

@larrybotha ,
Thanks!
Does It require oAuth2? Sorry, I didn't use twitter.

haha ye - I spent a couple hours wondering why I couldn't get a user account to authenticate with Twitter, only to realise they use OAuth1 for user authentication, and OAuth2 for app-only authentication.
Your commit looks good - assuming grant_type= client_credentials is added to the body of the request, which is what Twitter expects for app-only auth. I'm still new to HTTP, so I'm not really sure if setting params is the equivalent to defining the payload... I thought they were different things.

I thought they were different things.

I thought so. I tried to be consistent.
The default setup the way of the main flow https://github.com/oshliaer/apps-script-oauth2/blob/add-client-credentials-grant-flow/src/Service.gs#L323-L325
if (token && token.refresh_token) {
  try {
    this.refresh();

...

Service_.prototype.refresh = function() {
  grant_type: 'refresh_token'

I add the ability to choose a new path (a switcher) https://github.com/oshliaer/apps-script-oauth2/blob/add-client-credentials-grant-flow/src/Service.gs#L337-L339
} else if (this.params_.grant_type === 'client_credentials'){
  try {
    this.requestCCG_();

...

Service_.prototype.requestCCG_ = function() {
  grant_type: 'client_credentials'

Perhaps it needs to be a more transparent implementation? Or more unambiguous?
@erickoledadevrel

Any idea when this may get added? I am trying to use this with Centrify which also uses the client credentials flow https://developer.centrify.com/docs/client-credentials-flow

@bigben386 , It seems they want you to set an Authorization header on access token requests.
Please, look at Advanced configuration for this.
This part of the code still works fine but you have to resolve conflicts in the repo yourself.

I am using the authorization header. This is my code
function getCentrifyService(){   return OAuth2.createService('Centrify')   .setTokenUrl(TOKEN_URL)   .setClientId(CLIENT_ID)   .setScope(SCOPE)   .setClientSecret(CLIENT_SECRET)   .setPropertyStore(PropertiesService.getScriptProperties())   .setTokenHeaders({   'Authorization': 'Basic ' + Utilities.base64Encode(CLIENT_ID + ':' + CLIENT_SECRET) }); }
I am not getting a token when running getAccessToken(). I get "Error: Access not granted or expired. (line 413, file "Service", project "OAuth2")"
I am using these exact settings in Postman with oauth2 client credentials grant type and do not have any issues. Maybe I am missing something obvious. Any help is appreciated.


No property, bindable property, or event found for 'BorderColor'

I am using the latest version of Xamarin Forms, and all Nuget packages and I am building for Android Oreo 8.x When I add the following XAML I get errors for both BorderColor and FillColor indicating that neither is a bindable property:
XAML
<shapes:Circle>    <shapes:Circle.Paint>       <shapes:SSPaint           BorderColor="Blue"           FillColor="Red"            BorderWidth="1" />    </shapes:Circle.Paint> </shapes:Circle>
Error Message
Severity Code Description Project File Line Suppression State Error Position 196:53. No property, bindable property, or event found for 'BorderColor', or mismatching type between value and property.	PlanitUX.Client.Forms	C:\Dev\PlanitUX\PlanitUX\PlanitUX.Client.Forms\Views\TestPage.xaml	196
I have downloaded the project source code and successfully run the Android test app using these properties. However, Nuget Manager reports 24 packages that are out of date. Running Nuget Update does not complete  and fails with the following message:
Severity Code Description Project File Line	Suppression State Error Could not install package 'Xamarin.Android.Support.Animated.Vector.Drawable <IP_ADDRESS>'. You are trying to install this package into a project that targets 'MonoAndroid,Version=v8.0', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.
Any ideas?
Michael

I have the same problem with BorderColor, CardColor and IconBackgroundColor in my CardView. Did the tutorial on https://learn.microsoft.com/en-us/dotnet/maui/user-interface/controls/contentview?view=net-maui-8.0 (ContentView). Intelisense picks up the properties, but gives the same error when running the App.

I just found the issue. In my .cs file I used System.Drawing.Color instead of Microsoft.Maui.Graphics.Color.
Changing the tipe to Microsoft.Maui.Graphics.Color fixed the issue.


add files for composer support

Adds composer.json to be able to install your plugin through composer.

@hlohaus thx for the merge. Would be great to have tags/releases on github as well so they could be referenced instead of pointing to the master branch


gh-249: Replace azure with cloud in _pystats.yml

In order to run _pystats job with other non-Azure self-hosted machine.
Fixed https://github.com/faster-cpython/bench_runner/issues/249.

Can you also update the compare script to exclude these results?
https://github.com/faster-cpython/bench_runner/blob/1592e16a6921a6428a7b2812aa2e0245a806342c/bench_runner/scripts/compare.py#L157

Sure, I just made the changes! But I'm also wondering why we need to exclude these results from cloud machines?

Sure, I just made the changes! But I'm also wondering why we need to exclude these results from cloud machines?

Because when you collect pystats, the timing results are nonsensical, so doing a comparison on those timings wouldn't make sense.

Thanks for the contribution!


Install prerequisites

Hello,
I am trying to install substrate prerequisites using this command
 curl https://getsubstrate.io -sSf | bash -s -- --fast
but I got this error
Package clang is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

Package libclang-dev:i386 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'clang' has no installation candidate
E: Package 'libclang-dev:i386' has no installation candidate

I installed clang
sudo apt install libclang1-7
but it gave the same error.

I think the issue is because it is looking for libclang-dev:i386, so you need to make sure that the specific dependency is present on your system.


[Client] develop C# client

will it use HZ C# client lib?
or, just generate it via IKVM: http://www.ikvm.net/

Done


The activation was never quantized in the code

In the qlinear.py there are a lot of code snippets like this:
input = self.input_quant_fn(input) # activation quantization
input = self.move_aft(input) # add a FP bias
out = nn.functional.linear(input, weight)
or:
input = self.input_quant_fn(input) # activation quantization
input = self.move_aft(input) # add a FP bias
out = nn.functional.conv2d(input, weight, self.bias, self.stride,
self.padding, self.dilation, self.groups).
My point is: after you quantized the input to n-bit, a FP bias is added on it. Then what is the point of the input quantization??? How it can be activation quantization???? seriously? ICML?

@temporydan, if we have low-bit weights (W) and low-bit activations (X) along with a floating-point bias (FP_bias), it is possible to calculate the product of W and FP_bias separately before performing inference as W * X + W * FP_bias. This still allows us to accelerate the computation of W * X when deploying the quantized model. This practice is commonly used in Quantization-Aware Training (QAT). For more details, you can refer to the paper at https://arxiv.org/pdf/2003.03488.pdf. In the future, please make sure to do your research before leaving comments.

Thank you so much for your comments. However, can you please show us how this bias can accelerate the Conv op? Conv(x+bias)!=Conv(x)+Conv(bias). In practice, this added bias is meaningless on the Conv op. Or you may say we can use im2col to hijack the Conv op. Have you tried? GEMM+im2col is much slower than the Conv op.  Besides, you need to store a float16 copy of the unrolled bias for the input x. Using these tricks to improve the accuracy is meaningless.


Add support for isolation forest in variant QC

Can this be closed ?

Closing the pull request. We will keep the branch so this can be explored later


When using WKWebView in CocoaScrollView, has scroll issue

When using WKWebView in CocoaScrollView, if the content height of WKWebView is too high, CocoaScrollView will not start at the beginning of WKWebView.
    private func setupScrollView(outerGeometry: GeometryProxy, totalContentHeight: CGFloat, isContentLargerThanScrollView: Bool) -> some View {
        CocoaScrollView(.vertical, showsIndicators: true) {
            ArticleTopView(
                title: currentArticle.title ?? "Untitled",
                feedTitle: currentArticle.feed?.title ?? "Unknown",
                publishDate: currentArticle.publish_date ?? Date()
            )
            .onPreferenceChange(ViewHeightKey.self) { topViewHeight = $0 }

            WebView1(
                contentHeight: $webViewContentHeight,
                htmlString: currentArticle.html ?? "",
                goToPreviousArticle: goToPreviousArticle,
                goToNextArticle: goToNextArticle
            )
            .environmentObject(viewModel)
            .frame(height: webViewContentHeight)

            Spacer(minLength: outerGeometry.size.height / 5)
        }
        .onDragEnd {
            handleDragEnd(totalContentHeight: totalContentHeight, scrollViewHeight: outerGeometry.size.height)
        }
        .alwaysBounceVertical(true)
        .contentOffset($offset)
        .navigationBarHidden(true)
    }

https://github.com/SwiftUIX/SwiftUIX/assets/29099863/68b3270a-7c45-489b-9a3f-79eb797496d5

@xukecheng hmm... WKWebView is a scrollable view itself, I'm wondering whether this is a SwiftUIX bug or whether it should be out of scope. Fix would be write WKWebView subclass to give the web view an intrinsicContentSize by using JS to measure the content height?


Secure connection status in browsers via DNSSEC validation

Is your feature request related to a problem? Please describe.
We currently show that any content loaded via DNSLink is insecure - which is true.

Describe the solution you'd like
It would be nice if we could check the DNSSEC signature for the DNSLink entry to make sure we load the right content before using it to load anything via IPFS.
When there's a DNSSEC signature and it's been verified we should inform the browser that the context is now secure - as we have verified that this is indeed the right content for the domain.
Alternatives
If we use plain text DNS requests and don't do any signature verification OR the domain has no DNSSEC information stored, we don't have any security for the content similar to a page that gets loaded over HTTP.
It would be nice if we could do at least secure DNS requests for any DNSLink request until DNSSEC validation has been added. But I think it's worth a discussion if we should upgrade the context to secure in the browser for this case. Maybe we could configure two endpoints for secure DNS requests and if both agree we upgrade the context - while showing that this verification was only done by secure DNS requests, not by validating the DNSSEC signature.

@RubenKelevra We'll talk about this in more detail in our next weekly triage - thanks for raising.

@RubenKelevra We'll talk about this in more detail in our next weekly triage - thanks for raising.

@RubenKelevra this valid request, but is Brave-specific: Companion is unable to control those parts of browser UI.
Mind filling  this issue again in https://github.com/brave/brave-browser/issues/ + mention me there? :pray:  Thanks!

@RubenKelevra this valid request, but is Brave-specific: Companion is unable to control those parts of browser UI.
Mind filling  this issue again in https://github.com/brave/brave-browser/issues/ + mention me there? :pray:  Thanks!


Wrong owned item count?

is this an anomaly or how can we get 294 items from total of 98?

but show

kodadot/stick#169


Support for envFile property.

This is kind of a nice to have, but would serve as an alternative to entering the environment variables manually into the "env" property. envFile has been in the official C++ extension for a couple years, but is simply a .env file with series of key=value pairs. One could create a .env file manually this way, but more importantly, a build system could generate one that the debugger could read.
"envFile": "${workspaceFolder}/build/Debug/run.env"

run.env could be platform specific, and have changes to Path/PATH, LD_LIBRARY_PATH, DYLD_FALLBACK_LIBRARY_PATH, etc.

Nevermind... duplicates #506


[frontend] add reusable component PushDrawer

What changes were proposed in this pull request?
New PushDrawer component
Stateless* drawer that pushes main content to the side when open. Uses render props for drawer header, drawer content and main content. Supports smooth animations for drawers on the left and right side that can be used simultaneously.
The new Hue graph panel will use a push drawer (see UI design). Neither cuix or Antd offers such a component so I have built one based on our needs. We can continue to build on this component when we revamp the editor and need to replace the old side panels.
*) It does use an internal state to trigger change when CSS transition finish
Example usage:
import PushDrawer, { DrawerHeader} from './PushDrawer/PushDrawer';
const [isOpen, setIsOpen] = useState(true);

<PushDrawer
  leftDrawer={
    {
      content: () =>
 I'm the left drawer content

      header: () => <DrawerHeader onClose={() => setIsOpen(false)} title={'My drawer'} />,
      isOpen,
      width: '100px',
    }
  }
  mainContent={() =>
I'm the main content
}
/>

Additional changes:

Adds new unit tests
Adds classnames library for conditionally joining classNames together
Creates a hue5-globals.scss with styles as classes for common HTLM elements based on Cloudera Design Language. For convinience over using sass variables.

How was this patch tested?

Manual testing
New unit tests

Please review Hue Contributing Guide before opening a pull request.

Neither cuix or Antd offers such a component so I have built one based on our needs.

Does it make sense to move it cuix? as this is not available there.

Neither cuix or Antd offers such a component so I have built one based on our needs.

Does it make sense to move it cuix? as this is not available there.

Yes I think it makes sense. It will require a little rework (using less etc) but we should definitely look into that once we can use cuix. We are currently still waiting to get an approval from legal to do that.


Can't register plugins with flake8 / stevedore

flake8 / stevedore use the general entry_points feature to register plugins
For example
    entry_points={
        flake8_entry_point: [
            'X = flake8_example:ExamplePlugin',
        ],
    },

See http://flake8.pycqa.org/en/latest/plugin-development/registering-plugins.html

Maybe something like this could be a good notation:
[entry-points] # Optional super table

[entry-points."flake8.extension"]
X = "flake8_example:ExamplePlugin"
X101 = "flake8_example.ExamplePlugin"

What do you think?

This is now supported (version 0.4.0)


Would rewriting blocks work too?

Reading through some of the information here on this project I began to contemplate if there might be in any utility in doing something like using dd to read and rewrite each individual block in a file to accomplish the same mission intended by your code?
If so, this would mean that instead of needing enough space to hold the largest file in your directory, you would just need one free block worth of space. Moreover, one could set a number of blocks (if this idea would work), like read 10 blocks write 10 blocks, as it walked through the entire file.
Stuart

Certainly sounds like an interesting idea. Not sure how practical this is using a bash script. If you try out something I would be more than happy to link to your repository :)

Closing this for now, if you come up with something, let me know! 🤓


As a smart user, I would wish to put aliases to the commands available in the application

so that I can use the app much more efficiently

Implemented with #230


{SQL} az sql db import: improve help message

Improve the import help message. This fixes #14646
Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

/azp run


User is created if I cancel out of Add Account after error

Rancher Version: version master 11/10 UI 1.2.21
Steps to Reproduce:

Enable local authentication
Create an account with name Test
Create another account with name Test
After the duplicate name error, cancel out of the ADD account

Results: There is a account with no name that was created.
Expected: Should not be an extra account

Version - master 7-27
Now I can't create any users, every new user I try to add is a duplicate.

Version - master 2.0  7/31
Verified fixed


MSPSDS-6: Correctly throw error on ClamAV problem

There are a couple of things going on here:

The first is that the default Clamby config value for error_clamscan_client_error is false. That means if clamd has crashed (which I think is what's causing the issues on PaaS), Clamby.safe? returns false, we identify the file as a virus and delete it. It should now throw an error, which causes the job to be retried later (giving us time to restart the worker or something like that).
When we detected (or incorrectly detected, due to the above) that a file was a virus, we'd delete it. We deleted it synchronously using .purge, which I think meant that the currently running analyser would fail when it tried to set the metadata (on the now deleted blob). This was the Can't modify frozen hash error which we've been seeing a lot on the worker / sentry. We now .purge_later which I think should give the analyser enough time to set the metadata before it's deleted.

I'm planning to raise another ticket to deal with the UI aspects of a file detected as a virus. At the minute, you'll get a variety of view and controller errors if you upload a virus.

@raamSoftwire This might be related to your problem?

I've updated the PR to merge into staging. We probably want to include this in the next prod release.


Added Saitek P-Something gamepad.

Pretty old one but very usable.

Mapping has been added. Thanks for the PR!


feat: Add annotation propagation from Gateway to Load Balancer service

Now annotations from Gateway prefixed by prefixes defined in config will be propagated to Service. I followed actual ingress controller implementation for such feature and made it similar way
Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[X] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[X] Are you a user of Cilium? Please add yourself to the Users doc
[X] Thanks for contributing!

Fixes: #25357
Annotations prefixed by `gateway-lb-annotation-prefixes` flag value will be propagated to underlying Load Balancer Service

I'm not sure where to update documentation about annotation propagation. I guess I need to reflect it in similar way as in Ingress controller annotations page
Additionally, cloud-provider specific annotations for the LoadBalancer service
are supported.

By default, annotations with values beginning with:

* ``service.beta.kubernetes.io``
* ``service.kubernetes.io``
* ``cloud.google.com``

will be copied from an Ingress object to the generated LoadBalancer service objects.

This setting is controlled by the Cilium Operator's ``gateway-lb-annotation-prefixes``
config flag, and can be configured in Cilium's Helm ``values.yaml``
using the ``gatewayAPI.gatewayLBAnnotationPrefixes`` setting.

Please refer to the `Kubernetes documentation <https://kubernetes.io/docs/concepts/services-networking/service/#loadbalancer>`_
for more details.

Thanks for this PR, @sergeyshevch, but I'm afraid that we can't accept it - if we add annotation copying for Gateway now, it will stick around forever, even once we build something better. For this sort of thing, annotation copying is a hack that was necessary for Ingress, but now that we have structured config available in Gateway API, we can do this better.
I'm literally working upstream at the moment to try and get this finished, see https://github.com/kubernetes-sigs/gateway-api/pull/1757 for the GEP work.

As a result, I'm going to close this one. Thanks again for the effort.

That's quite sad because the lack of annotation propagations produces ugly hacks with mutation policies to set up the gateway correctly.
On the other hand, I totally support you in the upstream work and it's a better approach to make this feature.
Maybe we can implement it in any way now while looking forward to upstream Gateway API spec and later switch our implementation to structured config

I'm working on getting it to implementable before Gateway API GA - which will be in late October. I understand having to wait sucks, but the alternative is that we do something short-term now, and then spend years moving everyone to the better solution, which also sucks.


Circular view is not showing all of user uploaded file

Go here:
https://www.patricbrc.org/view/Genome/520456.3#view_tab=circular
and then upload the attached file.  You will find that it only shows the genes on one contig, and on none of the others.  Very frustrating.
BNWLFG2_S5 start stop for circular view.txt

Need to handle the extra header line  in the user file.

fixed and made a pull request.

Is this on alpha?  I tried it and am still getting the same error.
Rebecca
On Wed, Feb 14, 2018 at 3:43 PM, maovt<EMAIL_ADDRESS>wrote:

fixed and made a pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PATRIC3/patric3_website/issues/1885#issuecomment-365739076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK8ZdNI_QSWfZ7F5MOERhv8X8ENmjys5ks5tU0WBgaJpZM4SEWCL
.[image: Web Bug from
https://github.com/notifications/beacon/AK8ZdNP-L9CW4UOJWhaq-E50CcpJchp8ks5tU0WBgaJpZM4SEWCL.gif]

Should be available in the next build

@maovt please double check on production. we deployed yesterday.

Tested and verified on production.


Saving user's state for cultural and language searches

Saves user's selections rather than resetting them each time the user returns to an already-selected variable
Fixed badgeValue updates

Current coverage is 90.46%

Merging #234 into master will not affect coverage as of edffe41

@@            master    #234   diff @@
======================================
  Files           25      25
  Stmts         1667    1667
  Branches         0       0
  Methods          0       0
======================================
  Hit           1508    1508
  Partial          0       0
  Missed         159     159

Review entire Coverage Diff as of edffe41

Powered by Codecov. Updated on successful CI builds.


Update addons, for games like WoW

@hzeroo How hard would it be to implement addon updating?  Blizzard has released classic wow, and I would love to have an automatic solution to updating my game addons, rather than having to download them manually. when twitch merged with curse, the client never worked under wine, so was hoping for a native linux solution.

I don't know how hard would it be, i suspect it might not take too much time despite of not knowing too much about WOW addon ecosystem. The problem here is that one of the main objetives i wanted to address with this project appart from linux support is compatibility with old or low-powered systems, keeping the app as small and fast as I can archive, even it being an electron app, so adding extra features not directly tied to twitch viewer is out of scope for the project.
That being said, i'm not against a fork for a more feature-rich project, or even a plugin system built into the project that could provide extra features. I will focus my limited time on 'basic' features currently still not implemented such as VOD's, but i would consider any PR or proposal about this topic if there is interest on this being implemented.


Add support for deleteActivity and updateActivity

Adding support for these methods in WebChat will bring support for these methods to the emulator, which is important to allow developers to truly test their applications which may rely on these methods!
According to @compulim this would involve a few lines here:
https://github.com/Microsoft/BotFramework-WebChat/blob/master/packages/core/src/reducers/activities.js#L83

Before adding support, let's please discuss the design -- we added messageUpdate and messageDelete for this but the behaviors can be tricky so we should make sure we're thinking about them the same way.

@seaen tagging you for tracking purposes.

@benbrown Could you let us know the priority level for this? :)

It's not a high priority item -- it just presents an awkward problem for the SDK samples and demos because we can't show these APIs (which are present in the SDK) in use in the emulator.

@dandriscoll do you have any docs/specs pointing to messageUpdate/messageDelete activity type?
@benbrown do you have any end-to-end stories for this feature? It will help us to prioritize and give us insights on how messageUpdate/messageDelete should behave.

Do you have any news on this?

@DaKevinH No news.
@dandriscoll do you know who should involve in this work? And probably this issue should move to botframework-services for greater visibility.

Can you set up a meeting to discuss?

I'll move this to botframework-services in the meantime. @compulim ack'ing Dan's request for a meeting to discuss. Thans!


Build a Pluggable Web Widget: Part 2

In Chapter 3.5 Adding Accessibility
in the first Step: the user should add in "TextBox.tsx"
id={this.props.id}
but this.props.id does not exist - it must be
this.props.name

Hello Heiko,
Thank you for commenting on this issue for us. Our developers are currently working on the issue and a fix is forthcoming. I will update this issue when their fix has been published.
Connor

Hi @HeikoOsigus,
The Pluggable widgets tools v8.18.0 now includes the id for properties marked as labeled.
Please take a look at -> https://github.com/mendix/widgets-resources/releases/tag/pluggable-widgets-tools-8.18.0
Regards
Diego


Error: Cannot send value to non-payable function

I submitted this issue a few days ago and it was closed with a status of resolved.  However, when I log into Maker Market to deposit ETH into W-ETH I am still given this same error.  It looks like other users in Reddit are still getting this same error:
Error: Cannot send value to non-payable function
I look forward to a quick resolution so I can get back to purchasing more MKR.
Thank again in advance.

This issue is resolved, the changes are still not deployed.
One workaround is to simply send ether to the wrapper with sufficient gas, as the default function works like deposit: 0xecf8f87f810ecf450940c9f60066b4a7a501d6a7
PLEASE do a test transaction first with 0.1 ETH and 3m gas. If this does not correctly process the deposit (ie if it throws an exception), it means I'm wrong about the above - in that case, you have a more complex workaround: use MyEtherWallet or anything else that lets you manually craft calldata. The function signature is deposit(uint256), but the fallback function also processes

When do you expect this fix to be deployed?


Polymer Component Communication

Hello,
One challenge I find with Polymer is in component communication model. Often times I have elements nested 3 or more levels deep. Especially when laying out components, I run into problems passing messages between components.
In Angular, I have access to a ($broadcast)[https://docs.angularjs.org/api/ng/type/$rootScope.Scope#$broadcast] method to send messages to children elements. There is also the ($emit)[https://docs.angularjs.org/api/ng/type/$rootScope.Scope#$emit] method to send messages up the chain.
From what I can tell, Polymer has listeners. However, listeners only work with the immediate parent. In addition, there isn't a good way to send a message down the tree. The only way I could figure out is to use a DOM selector and then call a method on a component that way. Is my understanding correct or is there something I'm missing. The Angular approach seems easier to work with.

Maybe Pub/Sub could help you.

I wrote a lengthy article on approaches for communicating between elements:
https://www.polymer-project.org/0.5/articles/communication.html
It's targeted at Polymer 0.5, but the ideas are still sound. As @LM450N points out, for 1.0 https://elements.polymer-project.org/elements/iron-signals exists for pub/sub.

Closing as this is a question and not an issue.


Annotations sections in Download modal is showing up when opening it from Tomograms table

Describe the bug
Annotations text in Download modal should not be displayed when opening it from Tomograms table
To Reproduce
Steps to reproduce the behavior:

Go to https://cryoetdataportal.czscience.com/runs/364?table-tab=Tomograms
Click on Download link in tomograms table
See error

Screenshots
https://github.com/user-attachments/assets/dcc07166-27bb-4f86-93c3-2ee677968182
Desktop (please complete the following information):

OS: Sonoma 14.7
Browser Chrome

@Janeece i left a comment in the designs to confirm. it does seem out of place to download all annotations when we're specifically clicking to download a single tomogram

RE reported bug: Implementation is correct -- this was an in-flight change to address not having standardized annotations, sorry it wasnt (design change is here). The callout text aligned on by the team was universal for all download options (excluding "download dataset").
Please note that the original design's section cover sheet was labeled "archived" and a link to the new design is linked.

@Janeece i left a comment in the designs to confirm. it does seem out of place to download all annotations when we're specifically clicking to download a single tomogram

@junxini I think this is different than the reported bug? As i understand, the bug is about the callout text. Your comment seems to be about our approach to use the existing download flow instead of creating a separate flow for downloading individual tomograms from table — happy to discuss why we went this route and if we should log a design request ticket if thats the case!


Center in navbar is broken in material design

Hi,
Center in navbar is broken in material design. It is moving and not keeping in the middle of the navbar.
Here is an explanatory screenshot from http://www.idangero.us/framework7/docs-demos/navbar-multiple-links.html (same code, using a material design instead of ios one):

Did I miss something or is it really a bug?

Can't see such issue. What is your device/os version and app settings?

I am using the exact page from http://www.idangero.us/framework7/docs-demos/navbar-multiple-links.html except that I changed "ios" to "material" in the CSS includes. Everything else is the same.
I am viewing it in Firefox for desktop, in 640x480 with the responsive view.

You also have to enable material: true in app settings


Remove duplicated libraries and add the UPB library

With these changes we are able to build and run our library using the 1.26 version of gRPC.
The duplicate grpc libraries was causing the following error in the CMake:
    CMake Error at build/conanbuildinfo.cmake:1137 (add_library):
      add_library cannot create imported target "CONAN_LIB::grpc_grpc++" because
      another target with the same name already exists.
    Call Stack (most recent call first):
      build/conanbuildinfo.cmake:615 (conan_package_library_targets)
      build/conanbuildinfo.cmake:1019 (conan_define_targets)
      CMakeLists.txt:10 (conan_basic_setup)

    -- Library grpc_unsecure found /home/kelannen/.conan/data/grpc/1.26.0/kevin/testing/package/491e190cb39b1a21779584cb6f2e67fb2b1d8aab/lib/libgrpc_unsecure.a
    -- Library grpc_plugin_support found /home/kelannen/.conan/data/grpc/1.26.0/kevin/testing/package/491e190cb39b1a21779584cb6f2e67fb2b1d8aab/lib/libgrpc_plugin_support.a
    -- Library grpc_cronet found /home/kelannen/.conan/data/grpc/1.26.0/kevin/testing/package/491e190cb39b1a21779584cb6f2e67fb2b1d8aab/lib/libgrpc_cronet.a
    -- Library grpcpp_channelz found /home/kelannen/.conan/data/grpc/1.26.0/kevin/testing/package/491e190cb39b1a21779584cb6f2e67fb2b1d8aab/lib/libgrpcpp_channelz.a
    -- Library grpc found /home/kelannen/.conan/data/grpc/1.26.0/kevin/testing/package/491e190cb39b1a21779584cb6f2e67fb2b1d8aab/lib/libgrpc.a
    CMake Error at build/conanbuildinfo.cmake:1137 (add_library):
      add_library cannot create imported target "CONAN_LIB::grpc_grpc" because
      another target with the same name already exists.
    Call Stack (most recent call first):
      build/conanbuildinfo.cmake:615 (conan_package_library_targets)
      build/conanbuildinfo.cmake:1019 (conan_define_targets)
      CMakeLists.txt:10 (conan_basic_setup)```

Our library was also having an issue at the final linking stage without the UPB library so I have added it in as well.

Thanks for pointing out this issue!

Thanks, @Croydon for getting this fixed.
In the future I would prefer if my PR was merged or feedback was given for why it could not be merged instead of reauthoring the commits in your name. I am always happy to rebase or tweak a commit if it is needed.
Thanks,
Kevin

@kevswims You are right, sorry. I should have at least added you as a co-author
Main reason was that I merged first #29. I think that the order of the linking makes more sense there. And since I only checked the change summary of your pull request, I didn't realize that I could cherry-pick a8683dd separately.
Since I didn't merge it in any stable branch yet, I have rewritten this with a push force
https://github.com/inexorgame/conan-grpc/commit/a486769ddab127226d819f74de00facbb77f264a

Great! Thanks @Croydon


move jsmediatags to app/package.json

The jsmediatags is not packed in electron-builder because it requires a Two package.json Structure or asar disabled.
So sorry for that previous commit.  I guess I was self-defeating...

oh... it's ok. now we're clear about why.

oh... it's ok. now we're clear about why.

Would you be interested to use music-metadata to extract metadata?

Would you be interested to use music-metadata to extract metadata?


Can't resolve 'crypto' in micro-aes-gcm

https://github.com/textileio/js-threads/blob/21d2b2b96fe9d41ddcdb866beb61d4e48fe2bf56/packages/crypto/package.json#L38
Hi,
This dependency is causing a compilation problem in my Angular 9 project when the node typescript compilation target is the browser. Am I right in assuming this library is intended for the browser?
I get this error:
when starting up my app:
ERROR in ./node_modules/micro-aes-gcm/index.js
Module not found: Error: Can't resolve 'crypto' in '/inmanlabs/workspace/pnlp/pnlp-app/node_modules/micro-aes-gcm'

With a little digging, it looks like the following lead me to believe this is happening in any node Typescript compilation when the browser is an available target. The 'crypto' library is not available in the browser which means 'micro-aes-gcm:0.2.0' is not written as "Universal javascript".

https://stackoverflow.com/questions/54162297/module-not-found-error-cant-resolve-crypto/54162447
https://stackoverflow.com/questions/51087330/angular-6-many-cant-resolve-errors-crypto-fs-http-https-net-path-stream
https://github.com/iden3/snarkjs/issues/17

Unfortunately it looks like 0.2.0 is the latest version: https://github.com/paulmillr/micro-aes-gcm
Do you think I'm interpreting this correctly? Can you find a replacement library? Or poke paulmillr to use something other than the default 'crypto' in micro-aes-gcm?

Note, if you ever needed to reproduce (not that I'd expect you'd need to)
git clone https://github.com/pnlp-network/pnlp /tmp/issue-389
cd /tmp/issue-389
npm install
npm run build

rm -r /tmp/issue-389

Note, to work around this issue in the meantime, you can do the following:
add this to package.json:
  "browser": {
    "crypto": false
  },

add this pollyfill (in angular polyfills.ts):
// TODO: https://github.com/textileio/js-threads/issues/389
// remove once issue is fixed and we've upgraded to latest @textile/threads-core
(window as any).global = window;
(window as any).process = {};
(window as any).process = window;
(window as any).process.browser = true;
(window as any).process.version = '';
(window as any).process.versions = { node: false };
(window as any).process.nextTick = (() => {
  const canSetImmediate = typeof window !== 'undefined'
    && (window as any).setImmediate;
  const canPost = typeof window !== 'undefined'
    && window.postMessage && window.addEventListener;
  if (canSetImmediate) {
    return (f: any) => {
      return (window as any).setImmediate(f);
    };
  }
  if (canPost) {
    const queue: any = [];
    window.addEventListener('message', (ev) => {
      const source = ev.source;
      if ((source === window || source === null) && ev.data === 'process-tick') {
        ev.stopPropagation();
        if (queue.length > 0) {
          const fn = queue.shift();
          fn();
        }
      }
    }, true);
    return function nextTick(fn: any) {
      queue.push(fn);
      window.postMessage('process-tick', '*');
    };
  }
  return function nextTick(fn: any) {
    setTimeout(fn, 0);
  };
})();
(window as any).global.Buffer = (window as any).global.Buffer || require('buffer').Buffer;
// END https://github.com/textileio/js-threads/issues/389

Hi @dp-kb, glad to see you found a workaround. This is a bit strange to me though. micro-aes-gcm should be picking up whether you are in the browser or nodejs and using the appropriate APIs. Does Angular not provide access to the native browser crypto APIs? Maybe this is where it is tripping up: https://github.com/paulmillr/micro-aes-gcm/blob/master/index.js#L7

Hi @carsonfarmer -
Ah, good spot. Seeing those lines I would expect the crypto library to work in the browser (even in Angular). The error is happening in the Angular native dev server so Angular is probably doing some SSR ts-node compiling and then maybe making an assumption that the browser doesn't have crypto? Anyway... I will do some digging on this and pin down where this issue belongs (might be an Angular issue). Hold off on doing anything about this for the time being, like I said it's non-blocking anyway with the workaround.
Thanks!

Closing this as "no longer relevant" as we have pulled these deps out of the underlying library.


PS/2 keyboard adapter

Hello, I am running an old server repurposed to unraid (Supermicro MB X8HAD) and tiny pilot and see video output but keyboard does not work. There are ps/2 connectors and was wondering if would be possible to utilize them/has anyone had a solution for this.

Another project found a way to do it through PS/2, but TinyPilot's probably not going to go that route.
I'm experimenting with a new keyboard USB descriptor. Can you try running this command and testing to see if it's compatible with your motherboard?
export TINYPILOT_INSTALL_VARS="tinypilot_repo_branch=new-kb-hid"
curl \
  --silent \
  --show-error \
  https://raw.githubusercontent.com/mtlynch/tinypilot/master/quick-install | \
    bash - && \
  sudo reboot