Fix #2871 - Resolve blank map when running compiled with latest cordova-android (which uses AGP 4.1.3)

Fix #2871
From AGP 4.1.+ assert keyword/statements are enabled by the compile through code rewrite.
Which is causing PluginMap to fail

The issue is caused by PluginMap being initialised 2 times. The 2nd time it is initialised, CordovaPlugin.privateInitialize has assert to prevent this.
This PR removes initialising the PluginMap in CordovaGoogleMaps.getMap, because when it is added to PluginManager it will be initialised then.

NOTE
Although the relevant code in PluginManager and CordovaPlugin has not been changed for many years.
The plugin's codebase works for older cordova-android versions because they use older AGP.
In these older versions, assert keyword/statement do nothing

@ebhsgit I installed from your fork and I no longer get custom map marker icons. It always uses the default red pin from Google maps. Can you please test on your end to see if you have a similar problem?

The problem was that I was using a relative path ('./www/img/myimg.png'). When I switched to the full path (${cordova.file.applicationDirectory}www/img/myimg.png) then it worked. Of course the relative path was working well before I switched to this fork, so perhaps you could still check that too.

@CodeWithOz
Please note my fork is not intended for production usage.
That said, there shouldn't be any changes related to map markers in the fork.

I also have the issue with blank map. Works when AGP 4.0.2 is used, but is broken with AGP 4.1.x and above.

@wf9a5m75 please merge and publish a new version asap!

Kinda strange, because i don't see any changes that would cause that, but i can confirm what @CodeWithOz said. My Markers that are set via url: './assets/...' doesn't work anymore with your PR 🤔

I have not tested this fix but if it works please try to merge it so that we can use official release of this plugin and do not have to hack the solution ourselves in code. Without fix we are stuck to<EMAIL_ADDRESS>and can not use latest<EMAIL_ADDRESS>which needs Android Gradle Plugin (AGP) version 4.2.2. Thank you!

There are no changes in this branch related to markers.
This pr is branched from master.
Maybe you guys are using a different branch? eg: multiple_maps, there are lots of other changes between master and multiple_maps

Could you try the 20210804 branch?
I'm working on the next version recently, and don't touch v2.x these days. But it seems I've tried to improve something.

No i'm using your branch @ebhsgit 🤔 Before i was using Version 2.7.1.
@wf9a5m75 i think this should be fixed in a hotfix 2.7.1 as it blocks the migration to cordova-android@10

@wf9a5m75 i just tried to install from your branch. I got a lot of build errors, for example:
L.92: Cannot resolve symbol 'viewPlugins'
L.102: Inconvertible types; cannot cast 'org.apache.cordova.PluginEntry' to 'plugin.google.maps.PluginMarker'
L.410: Cannot resolve method 'getGoogleMap' in 'PluginMap'

in PluginMarker.java
and lots of Errors in CordovaGoogleMaps.java, which seems all be related to:
L.52: Cannot resolve symbol 'MyPluginLayout'

All in all 66 Errors

@CodeWithOz
@EinfachHans
Not sure what the issue is with icons.
The changes are simple, and are definitely not related to markers.

I am not experiencing any issues with markers

Do you add at least one via ./assets/...?

Yup.
eg:
./assets/imgs/map/markers/start.png

Do you just set the icon: './assets/imgs/map/markers/start.png' or do you set more options like size and anchor?

icon: {
   url: '....',
   size: '.....'
}

Strange, Version 2.7.1 works fine, but when i install from this PR directly it doesn't

https://github.com/mapsplugin/cordova-plugin-googlemaps/pull/2872#issuecomment-893316041
I was able to build the plugin in ionic v5 project at least.
Did you rebuild with command ionic cordova clean then  ionic cordova build android?

@wf9a5m75 I completely removed the platforms, plugins, www, node_modules folder and the package_lock.json and installed and build everything from zero

Okay i debugged a bit more, looks like this Marker Problem is cordova-android related: https://github.com/apache/cordova-android/issues/1316 - Thanks 😊

It really needs to be fixed soon, because the switch to Android 11 (API Level 30) will be required by Google from 1st Nov 2021. Can the current fix be used in production at least?

@wf9a5m75 any timeline on when this will be merged. Many of us are waiting for the fix  since switch to Android 11 (API Level 30) will be required by Google from 1st Nov 2021.
Could you please provide your thoughts upon this.
Thanks

@sachi097 you can also define it in the package.json:  "cordova-plugin-googlemaps": "git+https://github.com/mapsplugin/cordova-plugin-googlemaps.git#0b8ea76ad34fb2a202a9de1b9d0e051a82ad9443"
In my case, having a large project with lots of plugins, lots of features, custom markers, interactions:

The build passed,
The map is rendering properly
Partially tested but not in production

yes,

Did you get any build errors post install? - I musted add library com.google.android.libraries.maps:maps:3.1.0-beta in file project.properties
Yes
Did you test in production? - no

Are there any estimates when this will be merged?

Kindly Merge
thank you for the support :)

Kindly Merge
thank you for the support :)

Hi when can you merge this branch?

@alexp25 @lrobak0 thanks for your answers.
As per @lrobak0 comment installed the plugin via cordova plugin add https://github.com/mapsplugin/cordova-plugin-googlemaps#0b8ea76ad34fb2a202a9de1b9d0e051a82ad9443
Summary:

Got build errors initially, then uninstalled all the plugins and reinstalled (including https://github.com/mapsplugin/cordova-plugin-googlemaps#0b8ea76ad34fb2a202a9de1b9d0e051a82ad9443), deleted www folder and reinstall platform (ionic cordova platform add android@10.1.0). Post this build passed like a charm.
The map is rendering properly.
Currently, my app is in development phase. I will provide feedback once I test in production.

Map works like charm in the production.

Any updates on merging this? Just updated my cordova to android 10 and faced this issue. Testing the PR right now.

Fix #2871
From AGP 4.1.+ assert keyword/statements are enabled by the compile through code rewrite.
Which is causing PluginMap to fail
The issue is caused by PluginMap being initialised 2 times. The 2nd time it is initialised, CordovaPlugin.privateInitialize has assert to prevent this.
This PR removes initialising the PluginMap in CordovaGoogleMaps.getMap, because when it is added to PluginManager it will be initialised then.
NOTE
Although the relevant code in PluginManager and CordovaPlugin has not been changed for many years. The plugin's codebase works for older cordova-android versions because they use older AGP.
In these older versions, assert keyword/statement do nothing

Thank you , this fix Blank White Map Screen , but ,  now dont fire ,   plugin.google.maps.event.PANORAMA_READY  @wf9a5m75   Any help would be appreciated.

i have
"cordova-android": "^10.1.2",
"cordova-plugin-androidx-adapter": "^1.1.3",
"cordova-plugin-googlemaps": "^2.7.1",
google map not load display white screen
any one help me

Replace these files
CordovaGooleMaps
PluginMap
Copy here: ....<PROJECTNAME>\platforms\android\app\src\main\java\plugin\google\maps

From: subha-menuHuts @.>
Sent: Thursday, December 8, 2022 8:02 AM
To: mapsplugin/cordova-plugin-googlemaps @.>
Cc: Guillermo Franco @.>; Comment @.>
Subject: Re: [mapsplugin/cordova-plugin-googlemaps] Fix #2871 - Resolve blank map when running compiled with latest cordova-android (which uses AGP 4.1.3) (#2872)
i have
"cordova-android": "^10.1.2",
"cordova-plugin-androidx-adapter": "^1.1.3",
"cordova-plugin-googlemaps": "^2.7.1",
google map not load display white screen
any one help me
—
Reply to this email directly, view it on GitHubhttps://github.com/mapsplugin/cordova-plugin-googlemaps/pull/2872#issuecomment-1342623330, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIJPNTMDXNQGLISIN27V5IDWMHE5PANCNFSM45IT6X7Q.
You are receiving this because you commented.Message ID: @.***>


Link creation fails on minimal example?

I have created a gist with the relevant input and log files.
When I run mccortex with kmer size 3 with an input fasta containing two reads of length 4, and I then thread the same reads through the resulting graph, I do not get any links in the links file. Is this a bug, or am I doing something wrong?

For a repeat of length L you need a read of length L+2 to pair input and output edges. The shortest repeat in a k=3 graph is 3bp, so you need a read of 5bp covering it to generate links across it. Adding a base at the beginning should generate four links:
>0
GCGTT
>1
CCGTA

Thanks! That works.
A second question, since we're on this topic: Does mccortex annotate links in through bubbles or does mccortex only annotate through cycles? If not, why not?

Links are only added to tangles -- parts of the graph that collapse down then split out again. A simple bubble can be traversed without links if you try walking down all the options one at a time. I'll try to add some better documentation on links - sorry I don't have time right now. The paper on Linked de Bruijn graphs (or my thesis when I upload it) might be the best place to start.


[198] Generate and add "release notes" when distributing new builds on Firebase (staging and production)

Close #198

What happened 👀

Add echo "RELEASE_NOTE_CONTENT="$((git log -1 --merges | grep "\[") | grep . && echo "" || echo $(git log -1 --merges --format=%B))"" for staging flow (android and iOS)

Add echo "RELEASE_NOTE_CONTENT="$(git log --merges --pretty=%B $(git describe --abbrev=0 --tags)..HEAD | grep "\[")"" for production flow (Android only)

Insight 📝
N/A
Proof Of Work 📹

@doannimble the release note in the PoW contains the last commit message of this PR instead of the title of the PR. I found an issue and commented here. Please kindly check 🙏

@doannimble the release note in the PoW contains the last commit message of this PR instead of the title of the PR. I found an issue and commented here. Please kindly check 🙏

@manh-t Thank you for the great insight 👍
However, I just realized that Firebase will add the default release notes as the merge commit message or the working commit message already.
For example, from my working project:

when deploying a build from a merge commit on dev

when deploying a build from a working branch on a commit for testing

I doubt the last commit message from your POW comes from the built-in feature of Firebase CLI, not from your effort with the RELEASE_NOTE_CONTENT env, as I don't see its usage anywhere from the workflow 🤔  @doannimble

Falling back to the merge commit message or the working commit message in case the command can not get the PR title does not make sense here 💭  What do you think @doannimble @manh-t?

@doannimble please plan to have updates on this PR. Thank you 🤗

@doannimble Do you finish the necessary updates on this PR? Should we mark this PR to Draft?

@doannimble Do you finish the necessary updates on this PR? Should we mark this PR back to Draft?

@luongvo Yes I do. Please help to review and let me know if we need more improvement 🙏

@doannimble we need a new PoW. The current PoW is invalid 🙏

@doannimble is this still a work in progress?

@doannimble is this still a work in progress?

Yes @luongvo , I turned the PR to draft. When printing the log, it is empty. I'm finding another solution 🙏

@doannimble I believe we should drop all [Chore] Generate & update sample project commits when rebasing https://github.com/nimblehq/flutter-templates/pull/270/commits 🤓

Please update PR description to align to the new approach & POWs for both staging & production 🙏
@doannimble 🙏

Please update PR description to align to the new approach & POWs for both staging & production 🙏
@doannimble 🙏

@luongvo Updated 🙏


Cannot redeem invite on same machine as server

Hello,
I am trying to redeem an invite on the same machine as the server but I am getting a HTTP 401 error.
I had someone else redeem an invite on the server and it worked. Any ideas what's happening?
↳ inn --version
innernet 1.4.1

$ russ @ devmachine in ~ [18:26:31]
↳ sudo innernet-server add-peer testinnernet
✔ Eligible CIDRs for peer · devnet (<IP_ADDRESS>/24)
✔ IP · <IP_ADDRESS>
✔ Name · test
✔ Make test an admin? · no
✔ Invite expires after · 14d
✔ Save peer invitation file to · test.toml
✔ Create peer test? · yes
adding to WireGuard interface: test (xxxxxx)

Peer "test" added
Peer invitation file written to test.toml
Please send it to them securely (eg. via magic-wormhole) to bootstrap them onto the network.

$ russ @ devmachine in ~ [18:29:59]
↳ sudo inn install test.toml
[!] /etc/innernet is world-accessible (mode is 0o755). This is probably not what you want.
✔ Interface name · testinnernettest
[*] bringing up the interface.
[*] Generating new keypair.
[*] Registering keypair with server (at <IP_ADDRESS>:51820).
[E] failed to start the interface: http://<IP_ADDRESS>:51820/v1/user/redeem: status code 401.
[*] bringing down the interface.
[E] Failed to redeem invite. Now's a good time to make sure the server is started and accessible!

[E] http://<IP_ADDRESS>:51820/v1/user/redeem: status code 401

Also thank you for this project! Rust, wireguard, peer to peer, it has everything!

Hey @russkel and sorry for the late reply! One can't run a client and server on the same machine (or, more specifically, the same network namespace) since you'll have conflicting routes (ip route), as both interfaces will be trying to route <IP_ADDRESS>/16.
While you could probably do some fancy stuff with ip-netns, my general recommendation would be to have the innernet-server on a separate host.

Thanks @mcginty. I see. Thanks for the reply. If I were to run the server in a docker container that will solve the namespace issues?

The routes are already set up, right?
Isn't it enough to

let innernet-server also update the local /etc/hosts
change SERVER_NAME in innernet-server new to be customizable and default to the hostname?

@russkel running it in a docker container should also solve the namespace issue since it also has a similar network namespace concept :).

Thanks for the confirmation, and work on this project!
Showing a difference between clients and servers sounds nice. Are there other user-visible differences, aside from it having access to every sub-CIDR?
Maybe a boolean option to allow the server to participate? Would provide a natural spot for documentation.

+1 for an option for adding the server as a network participant.
In particular, I would like it to populate its own /etc/hosts and also show up under its own hostname instead of as innernet-server. If there's interest in those features, I may be able to contribute them

In particular, I would like it to populate its own /etc/hosts and also be reachable using its own hostname instead of just innernet-server. If there's interest in those features, I may be able to contribute them

Contributions welcome! It makes most sense to me add support for writing /etc/hosts also on the server and allow to specify its innernet name. (as opposed to joining as a peer).

This acually looks like a duplicate of #45?
I'm not sure about supporting joining as a peer from the server machine, but we can surely make the server name configurable and allow writing the /etc/hosts file from the server.


Update version contraints to allow installation in Laravel ^7.0

Duplicate
https://github.com/brozot/Laravel-FCM/pull/174
Furthermore, you are commiting a file with change in package's name to the another name of a fork repository.

@gpibarra whoops, my bad, didn't see that


implemented login and sign up pages

Authentication system with manual login/signup + Google Oauth setup is implemented.
Vitest is setup for unit testing.
Tests for home page have been written.


start on loop tile

Various issues to address with the loop tile

where can't we loop (switch page, anything else?)
looping reveals problem with field editor... need some basic mod to editor.ts to support selection between static tile and one with field editor (see looping around icon editor)

Maybe try a tile that filled with a solid color so it really looks like a wall. It look too much like an actuator now.

don't allow loop of assignments and radio send... don't really see the purpose. Mainly for repeating animations and sounds

seems to work for me; may need some more testing


Rework of Annotations on Android and iOS

I encountered, that Annotations (Polyline and Polygon) don't work. So I changed some parts.

MapView.Annotations is now a ObservableCollection and monitored from start
Annotation.Coordinates are a ObservableCollection and monitored from start
Polyline now has a StrokeColor, StrokeWidth and Alpha property
Polygon now has a StrokeColor, StrokeWidth (isn't used in Mapbox up to now), Alpha and FillColor property

Some things are missing up to now:

A click event for Annotations other than PointAnnotation (see Mapbox issue #9443)
Holes for Polygons
An InfoWindow for Polyline and Polygon

I don't know, if this is the correct way for this changes. If not, write a note and I will change, what isn't correct.
Thank you for your work.
PS: Mapbox 5.1.1 is out :)

IEnumerable isn't enough. Perhaps a IList is ok, because you want to Add and Remove things to this list. The property itself must be an ObservableCollection and should be readonly.
Why should Annotations not an ObservableCollection? And why should the developer set this property? If Annotations or Coordinates are of type IList, than you could do all things you want. You could Add, Remove and Clear the list. That's ok. Or not?

:). From the renderer, you don't need to add/remove or any other kinds of operation on property Annotations . The place you do those things is in your VMs, then you could use whatever kind of collection type you want for the real purpose.
Plz see ListView ItemSource for example.
Cheers.

Ah, I think I get it. You say, the developer is responsible, if she/he wants, that Mapbox tracks changes on the Annotations. If she/he uses an Array there isn't any tracking. If she/he uses an ObservableCollection, than Mapbox tracks any changes. Is this correct?

That's right. Our renderer will track the changes if the dev uses ObservableCollection. Otherwise, don't need.

I have to close your request due to its out-of-date comparing to our internal changes.
I tried to merge but I found you still use ObservableCollection even if I suggested to use IEnumerable.
Thanks for your effort.


irmin-graphql: Bump graphql-cohttp to 0.12.1

Waiting for https://github.com/ocaml/opam-repository/pull/13773 to be merged...

This error on CI seems to be out of the ordinary: https://travis-ci.org/mirage/irmin/jobs/512928463#L3876

That's indeed very weird:
# File "graphql_parser/src/graphql_parser.ml", line 1, characters 5-11:
# Error: Unbound module Result
#     ocamlopt graphql_parser/src/.graphql_parser.objs/native/graphql_parser.{cmx,o} (exit 2)

I made a mistake in the opam file of graphql_parser (https://github.com/ocaml/opam-repository/pull/13803). Can we try rerun CI?

CI is restarted.


Youtube iFrame API

I am embedding a youtube video in a slider as shown below

      // 2. This code loads the IFrame Player API code asynchronously.
      var tag = document.createElement('script');

      tag.src = "https://www.youtube.com/iframe_api";
      var firstScriptTag = document.getElementsByTagName('script')[0];
      firstScriptTag.parentNode.insertBefore(tag, firstScriptTag);

      // 3. This function creates an  (and YouTube player)
      var player;
      function onYouTubeIframeAPIReady() {
        player = new YT.Player('player', {
          height: '390',
          width: '640',
          videoId: 'M7lc1UVf-VE',
          events: {
            'onReady': onPlayerReady,
            'onStateChange': onPlayerStateChange
          }
        });
      }

      // 4. The API will call this function when the video player is ready.
      function onPlayerReady(event) {
        event.target.playVideo();
      }

      // 5. The API calls this function when the player's state changes.
      var done = false;
      function onPlayerStateChange(event) {
      //   SOME where here I need to open or close the Cover
      }
      function stopVideo() {
        player.stopVideo();
      }

And I have added jssor.player.ytiframe.js.
Now my question is how can I open or close the "Cover" within "onPlayerStateChange" function above.

Is it solved or irrelevant?

It can't work like this.
I am looking for better solution to embed youtube video.


⚠ Fortnite has degraded performance

In b21870f, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1556 ms

Resolved: Fortnite performance has improved in 80f5067.


UnregisterClass after DestroyWindow

Add UnregisterClass  after DestroyWindow (here).
Otherwise if I wrote a script like your example and exec it twice in python interpreter, there would be error messages saying that class already exist.
And that's because destroying the windows doesn't unregister the class.

Hi @openopentw
Sorry for the late response,I was caught up with work. Thank you for your interest in this project.
I've fixed the issue. Thank you for bringing this to my notice.


error: no ipv4-network or ipv6-network options set

I get this log after installation
Oct 27 11:48:23 server.domain.com systemd[1]: Starting OpenConnect SSL VPN server...
Oct 27 11:48:23 server.domain.com systemd[1]: Started OpenConnect SSL VPN server.
Oct 27 11:48:23 server.domain.com ocserv[23095]: note: skipping 'pid-file' config option
Oct 27 11:48:23 server.domain.com ocserv[23095]: note: vhost:default: setting 'pam' as primary authentication method
Oct 27 11:48:23 server.domain.com ocserv[23095]: error: no ipv4-network or ipv6-network options set.
Oct 27 11:48:23 server.domain.com systemd[1]: ocserv.service: Main process exited, code=exited, status=1/FAILURE
Oct 27 11:48:23 server.domain.com systemd[1]: ocserv.service: Failed with result 'exit-code'.

This was because I was trying to  cancel installation by Ctrl+C. This does not stop the installation and just cancels on of the commands. I used some of the commands in the script manually and it's fixed now.

What commands exactly? Is there any way to uninstall and reinstall clean?


my mods are not showing up

every time i get do install a new mod it don't show

same did u find a fix

ModMenu gets the list of mods from Fabric and by default displays all mods that are not marked as a library.
If you are adding a library and want to see it in ModMenu you can enable the display of libraries in ModMenu's settings.
If you are adding a mod that isn't a library and it isn't showing up in ModMenu it means that Fabric hasn't loaded the mod. Please consult the Fabric Wiki on how to add mods.


feat(accessibility): optionally disable aria-current tag

Some of the accessibility specifications require removal of aria-current tag in lists.

This tag is displayed on the currently focused <ListItem>'s.

For this, I am suggesting an ariaConfig prop which may contain a JSON value for tweaking accessibility properties.
Here:
const ariaConfiguration = {disableAriaCurrent : true};
<List ariaConfig={ariaConfiguration }>

Can be used to stop the population of aria-current = true tag.
TODOs

Write tests
Update Snapshots

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@andrewholsted  - We have more than a  dozen of aria-* attributes . link, I thought of passing a JSON object, since the behavior of placement of aria-current tag is handled by the momentum-ui code. In most other cases, the tags are static and dont change their values and/or location in the DOM.

@pauljeter - Yes. Will be updating,

@bfbiggs - Yes... In another usecase, I need (optionally?) specify aria-posinset tags...
I think for such tweaks in ARIA behaviour its better to use a single ariaConfig prop... which can be customized for configuring such behaviour...
Many similar use cases could be possible...

Added Test Cases
Updated Snapshots
Pipeline Passing
Added ariaConfig props to Lists and Menus


python310Packages.pymbolic: remove pytest

Description of changes
Fix build (https://hydra.nixos.org/build/176212143)
ZHF: #172160

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 173150 run on x86_64-linux 1

4 packages built:

python310Packages.loopy
python310Packages.pymbolic
python39Packages.loopy
python39Packages.pymbolic


Refactor asakusafw-organizer plug-in.

Added AbstractOrganizer class that is also used in Asakusa Spark Gradle plug-in.

test this please

BUILD SUCCESS.

LGTM.

BUILD SUCCESS.


Encrypted fields need a buttonLabel

Is your feature request related to a problem? Please describe.
We internationalize all of the strings in our UI using labels and we can't do that for the "CHANGE" button for encrypted fields.
Describe the solution you'd like
Seems simple enough to just add a buttonLabel
Describe alternatives you've considered
For now, we are just living with this one button being in English.

Hi @lfeigen
Thanks for your issue.
If I'm understanding correctly you're using the secret  input field?
It's indeed true that most of the labels are adaptable except the ones from the secret  input field.
Not only the 'Change' button, but also the dialog labels should be adaptable then if I'm understanding correctly?
When I have the some free time, I'll add the labels or - it this is a pressing issue - you can of course contribute and leave a PR.

I hope to be able to contribute changes on one or more of these issues I've raised.


Why IMGUI is so slow ?

Hello,
I compiled bgfx with target Win64 - Release. Launching the HelloWorld example appears a performance dialog. If you move the dialog with mouse you will see the lag ( mouse moves faster than dialog)

That's latency between hardware mouse, and rendering. It's up to you to hide it by adding software mouse one you detect dragging (applications like Chrome do this when window was dragged).
More appropriate place to ask Q like these is:
https://gitter.im/bkaradzic/bgfx

Some info how "hardware mouse cursor" works - https://stackoverflow.com/questions/48833250/why-is-the-mouse-cursor-drawn-faster-than-applications
And here is the code how to enable/disable software one https://github.com/beardsvibe/win32softwarecursor/blob/master/win32softwarecursor.c


Compiling Finch itself is slow.

The time to generate an implementation for the first tensor program is roughly 30s on my machine (due to Julia compiling Finch for the first time), but subsequent generation times are <1s. Further investigation needed.

After a quick conversation with Chris Elrod today, the following approaches were suggested:

Use an unityped IR during compilation check out Unityper.jl
Avoid the use of MacroTools

Another resource: https://sciml.ai/news/2022/09/21/compile_time/


#219 Duplicate Function GetForm

This PR seems to add unused functions, with no clear usage, we're closing it.


Investigate GHA to Check Key Vault Certificates Expiration failure

Getting the following error when trying to add the IP of the Github runner to the Key Vault firewall:
Message: The api-version '2023-02-01' is invalid. The supported versions are '2023-07-01,2023-07-01-preview,2023-03-01-preview,2022-12-01,2022-11-01-preview,2022-09-01,2022-06-01,2022-05-01,2022-03-01-preview,2022-01-01,2021-04-01,2021-01-01,2020-10-01,2020-09-01,2020-08-01,2020-07-01,2020-06-01,2020-05-01,2020-01-01,2019-11-01,2019-10-01,2019-09-01,2019-08-01,2019-07-01,2019-06-01,2019-05-10,2019-05-01,2019-03-01,2018-11-01,2018-09-01,2018-08-01,2018-07-01,2018-06-01,2018-05-01,2018-02-01,2018-01-01,2017-12-01,2017-08-01,2017-06-01,2017-05-10,2017-05-01,2017-03-01,2016-09-01,2016-07-01,2016-06-01,2016-02-01,2015-11-01,2015-01-01,2014-04-01-preview,2014-04-01,2014-01-01,2013-03-01,2014-02-26,2014-04'.

I am not seeing any failure. We are not getting notifications just because there no cert expiring in the next 30 days
the "add runner ip to firewall" step also runs successfully


Handling Non-Time Metric Aggregations

Problem Statement
We've now heard from multiple partners that sometimes they want to return a single number representing a metric. They don't need it to be listed by day, week, month, etc - they want the single numerical representation for that business concept.
And we can't do that right now. With time_grain and timestamp being required fields for a metric definition, we currently only allow for returned datasets to be broken down by some sort of time aggregation. The partners could hypothetically pull the most recent value for the metric but that only works in some cases. Cases where a multi-month aggregation, or even multi-year aggregation, of a metric cannot be handled with current functionality.
Feature Description
As I see it, there are two options as we move forward:

Re-architect the metric node to no longer require a timestamp and time-grain field. We would then change the logic in the metrics package to be able to handle when these are missing and still return the correct logic.

There still exists the possibility that sometimes a metric with a timestamp would be needed to aggregated over all time.

Add support for the all type in time_grains. This would aggregate over all time and return the metric value. However, this still has implementation questions

Do we return a date field with the metric? My gut instinct is no but there could be some shenanigans around what time range really constitutes all

In my opinion, the perfect solutions is a combination of the two of them. We should support all, or some alternative name, to allow for use cases that aren't focused around time-series data. Additionally, we should consider re-architecting the metric node to allow for non-timestamped metrics, which is an area we might run into.
How Does This Relate To The Vision
This is a great issue for us to have a broader discussion on what the future of the semantic layer looks like. Is it a permissive framework that gives users as much flexibility as they need or is it a tightly woven experience around an opinionated stance of what metrics are and the best practices associated with them?

Closing because I duplicated in #97 and that one has much better implementation information


npm-check fails due to calling npm ... --color=always

Describe the bug
when you have an npm-based project and run npx npm-check -u and update some dependency, it will run a command like npm install<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>--save-dev --color=always
This fails with an error from turbo. Removing the --color=always makes it work
Link to the blitz that caused the error
https://stackblitz.com/edit/qwik-style9?file=src%2Froutes%2Findex.tsx,node_modules%2Fstyle9%2Fsrc%2Fhelpers%2Fvalidate.js,vite.config.ts,dist%2Fbuild%2Fq-a8c09fbf.js
Steps to reproduce
npx npm-check -u
Expected behavior
it should accept common npm flags
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 112
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 223938441,
  "usedJSHeapSize": 205561441,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 9c984a48

Additional context
No response

:wave: Thanks for the report. This is a known limitation of turbo, our NPM shim.
We are actually looking into releasing "real" NPM as the default package manager real soon, which would actually solve this. I'll track this issue so we updated it when that happens.

Actually, I'm closing this in favor of https://github.com/stackblitz/webcontainer-core/issues/1070, so we centralize it a little bit.


Added option to exit after migrating the database

I added an option to exit after applying the migration to the database. This allows using the Docker image to be used as an init-container and to do the database operations separately from the hosting part. This is especially useful in kubernetes deployments where you want to make sure everything is setup before starting any other containers that start listening.

Thanks! I am sorry for delay. 😊


Where are Database initialisers like DropCreateDatabaseAlways?

I cannot find the database initialisers such as DropCreateDatabaseAlways?
The Database.SetInitializer seems to be removed as well. What's the best way to initialise database?
Thanks

Database initializers no longer exist in EF7. You can use DbContext.Database.EnsureCreated EnsureDeleted etc. in your startup code to achieve the same thing (generally most useful for testing etc.).


Feature/home page text

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


add real documentation

use wiki?

Hi @lily-mayfield , I'd be happy to help with any documentation! Is there currently anything I can assist in?


Input bar + Write message

Create input bar and be able to write a message and show in the conversation.

Depends on #400

Depends on #400


[WIP] Kai thesis draft

Extend Maggy to support asynchronous Bayesian optimization combined with budget allocation methods like early stopping or hyperband

superseded by #64


How to disable text edit in the FormBuilderTextField but enable suffix icon click?

Hi,
How to disable text edit in the FormBuilderTextField but enable suffix icon click?
Arnoldas

There seems not to be a way of doing this in Flutter's TextFormField. When enabled is set to false, suffix and suffixIcon are disabled.
You may consider having your suffix icon as a separate widget instead of being part of the FormBuilderTextField's decoration

Thanks for the answer. Will create separate icon widget.

Container(
child: Stack(
alignment: Alignment.centerRight,
children: [
FormBuilderTextField(
readOnly: true,
controller: _codeController,
attribute: 'qq',
),
IconButton(
icon: Icon(Icons.traffic),
onPressed: () {
print('qq');
},
)
],
),
),

Can be close this issue?

Can be close this issue?

Yes, you can close. Thanks.

Hi @danvick. I don't have time all days, but I can contribute in my free time. Please add my personal account @deandreamatias

Neither do I, that's why some issues take time before they are solved. 😌
Glad to have you on board 🥳


ObjectLink and Cbor Mapping

Hi,
I am trying to implement the Client and Leshan Device Management Server to support the ObjectLink and Cbor to be used in the Device ExtDevInfo or LWM2m Software Management.
If I perform a read on ExtDevInfo or Package Settings, I expect a Cbor byte array. What do I do to implement ObjectLink into Cbor encoded byte array.
Has anyone attempt to do anything to map ObjectLink into Cbor and Cbor into ObjectLink?
Please share and guide me with some examples.
Thank in advanced.
Regards, hulk
Device
ExtDevInfo
R
Objlnk

LWM2M Software Management
Package Settings
RW
Objlnk

Currently, we only support LWM2M 1.0 and if I'm not wrong Cbor is only in 1.1.
We are thinking about LWM2M 1.1, and this is not impossible we begin to work on it at mid term.
Anyway, Leshan is pretty flexible about encoder/decoder so if you want to add or replace default decoder/encoder, you can !
To do that, just implements a LwM2mNodeDecoder or LwM2mNodeEncoder, you can start from DefaultLwM2mNodeDecoder or DefaultLwM2mNodeEncoder.
Then you can add your custom encode/decoder using LeshanBuilder.setEncoder and LeshanBuilder.setDecoder.
About ObjectLink, I'm not sure to understand what is currently missing ?
Don't forget to share your work with us ;)

(about LWM2M 1.1, if you want to share your thought about it : https://github.com/eclipse/leshan/issues/563)

Hi Bernard - the Leshan Gatekeeper. Thank for taking your time to explain.
There're few OMA IPSO Object specifications documented with the ObjLnk type in which is implemented into the ObjectLink Class in Leshan.
I believe that ObjectLink in the IPSO Object-1 is used to specify the "Relation" with other IPSO objects. The IPSO Object-1 also contains the PayLoad Attribute documented with the byte array described the cbor encoding.
Regards, hlkn

So you want support of ObjectLink with cbor encoding ?
I understood that your request for objectLink was a separated feature request and I didn't understand because it seems to me that ObjectLink is currently implemented in Leshan (for json and tlv format).

Could we close this issue ?

Hi Bernard, as you explained above, Leshan is pretty flexible to extend encoder/decoder, that is approach what I did. In addition, DefaultLwM2MNodeDecoder/DefaultLwM2MNodeEncoder, knownContentFormat in ContentFomat.java, and client-detail.html in leshan-server-demo component also need to be updated. If I want to share my work, is there any strategy of introducing third party dependency in Leshan, because of CBOR encoder/decoder right now are using open source library.

You can start by opening a PR.
You will need to [sign eclipse CLA] (https://github.com/eclipse/leshan/wiki/How-to-contribute#legal-stuff-)

So we can close this issue and we will continue the discussion in the PR.


add a link in tootltip

hallo there
i try to add a html link inside a tooltip, is it should be simple, but i can't get it work fine.
this is the code, where am I wrong ?
	$('.ammessa-tooltip').tooltipster({  //tooltip per la parola ammessa nel codice
			contentAsHTML: true,
			timer: 2000,
			autoClose: false,
            content: $('Non sai cosa significa ammessa ?  Cerca nel regolamento '),
        });

thanks to everyone

I can't see an error here, must be something else. But please head over to Stackoverflow for implementation questions. Thank you.


Use ES6 import/export

Instead of require

Our refactor branch uses almost all the features of ES2015 :)


When will release the midi version

Hello @MoonInTheRiver ,
Thank you release the singing version!
When will release the midi version?

MIDI version is not the content of our paper (we never mention midi in any version of our paper). I write some midi-related codes just out of my interest.  Also, we do not have manually labeled MIDI for PopCS (thus I could only conduct related experiments on automatically transcripted midi...). People can try their datasets with MIDI by themselves and it must be very fun  :)

Ohhhhhhh, I've added it... https://github.com/MoonInTheRiver/DiffSinger/blob/master/usr/configs/midi/readme.md


[SPARK-15386] [CORE] Master doesn't compile against Java 1.7 / Process.isAlive

What changes were proposed in this pull request?
Remove call to Process.isAlive -- Java 8 only. Introduced in https://github.com/apache/spark/pull/13042 / SPARK-15263
How was this patch tested?
Jenkins tests

Test build #58793 has started for PR 13174 at commit 5315de5.

LGTM.

Test build #58793 has finished for PR 13174 at commit 5315de5.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Jenkins retest this please

Test build #58794 has started for PR 13174 at commit 5315de5.

Test build #58796 has started for PR 13174 at commit 5315de5.

Test build #58794 has finished for PR 13174 at commit 5315de5.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Definitely spurious, but happened twice. I think we should pull the trigger on this hot-fix pretty soon regardless to get Java 7 compilation working again in master. Although I've learned to never say never, I can't fathom how this change would break something.

@srowen : Sorry about this. In order to not let this happen again, are we going to do something ? If trunk needs to work with Java 7 or higher, then Jenkins tests should be ran with Java 7. In PRs, the Jenkins bot could then help catch such problems before PRs get merged into trunk. I recall that a month or two ago it used to be Java 7 but recently it seems to have switched to Java 8.

Yeah, this is really a bit of a test infrastructure issue. Ideally Jenkins' PR builders run with 7. We have a bunch of other jobs that test things like this, later. It may be a moot point soon enough (when Spark ~2.1 requires Java 8?) that we just accept the occasional hiccup like this. It's not a big deal.

Test build #2992 has started for PR 13174 at commit 5315de5.

The PR builders use java 8 because parts of the build need to run with target = 1.8 (to test closures, for example).
But the really correct way to do this would have been to use jdk 8 but use jdk 7's rt.jar when compiling with target = 1.7. (javac even prints out a warning when you don't do that.)
$ javac -target 1.7 -source 1.7 test.java
warning: [options] bootstrap class path not set in conjunction with -source 1.7

Yeah that's totally right. I reveal my bias, which is to just go ahead and require Java 8, perhaps soon in master, to resolve this.

Test build #2992 has finished for PR 13174 at commit 5315de5.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

@JoshRosen knows more about mima and might have seen these errors in the past?

Test build #58796 has finished for PR 13174 at commit 5315de5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Jenkins retest this please

Test build #58811 has started for PR 13174 at commit 5315de5.

So, this branch passes all tests for me locally. If it still faces some spurious failures, I'm going to merge it anyway.

Test build #58811 has finished for PR 13174 at commit 5315de5.

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

Merged build finished. Test PASSed.

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

Merging to master.

I'm on vacation and won't be able to do this myself, but if we want to avoid this problem in the future then someone should determine the path to a Java 7 rt.jar, modify the SBT and Maven builds so that the path to RT.JAR can be overridden via an environment variable, and update the Jenkins configuration scripts to set that new configuration variable. If nobody gets to this, I can take care of it when I'm back.
In addition, someone should configure Maven Enforcer to fail if we accidentally depend on JARs which contain Java 8 bytecode.


Validate tests

** You have to install Yarn and foundry to make it work**
Install foundry
This is the rust codes of validating tests in https://github.com/eth-infinitism/bundler/blob/main/packages/bundler/test/ValidateManager.test.ts .

The pr introduce foundry as a testing evm node which would provide enough function for bundler to interact with.
Include bundler repo
Introduces the validating test cases.
Enable clippy lint in tests.

This is good, will be very helpful :rocket:

Looks good @zsluedem. I wonder if these tests will stay relevant in the future or if we should focus on https://github.com/eth-infinitism/bundler-spec-tests. This was released two weeks ago and will be the official test suite for bundlers (maybe integrate running these tests instead in the repo).
What do you think?

Looks good @zsluedem. I wonder if these tests will stay relevant in the future or if we should focus on https://github.com/eth-infinitism/bundler-spec-tests. This was released two weeks ago and will be the official test suite for bundlers (maybe integrate running these tests instead in the repo).
What do you think?

Hm...I didn't know the test suite. I read the test codes there today and compare. They got very similar test contracts for opcode-banning parts.
The test suite would be run with a bundler client and an execution client from higher-layer integration perspective. I think we could have both because this could help you develop the simulation checking function (I think). You don't need to have a fully functional bundler to run the test suite right now.
But I am Ok if you want to just use the test suite.

LGTM :rocket:


feat: service discovery over Kubernetes WIP

cc @michalszynkiewicz
I finally push the tests using the fabric8 Kubernetes Server Mock in CRUD mode :-)

The only big issue here is the blocking nature of the client, the rest are rather notes/observations


WMI to CIM conversion

This code was originally written in vbs, then ported to PowerShell. At that time WMI cmdlets were standard. PSScriptAnalyzer informs me that it's time for a change.
File 'ActiveDirectoryManagement.psm1' uses WMI cmdlet. For PowerShell 3.0 and above, use CIM cmdlet which perform the same tasks as the WMI cmdlets. The CIM cmdlets comply with WS-Management (WSMan) standards and with the Common Information Model (CIM) standard, which enables the cmdlets to use the same techniques to manage Windows computers and those running other operating systems.
Affected lines:

782
784
793
795

This was resolved see b9ec8d594b2fe98c365d696fddc772f8ce0c10b6


ImportError: No module named munkilib | Munkireport-3.0.1 & 2

Unable to install munki client and ending up with below error while executing the following command:sudo /bin/bash -c "$(curl -s http://x.x.x.x:4433/public/index.php?/install)"
Error:
CHMM115523:~ sysadmin$ sudo /bin/bash -c "$(curl -s http://x.x.x.x:4433/public/index.php?/install)"
Preparing /usr/local/munki/
BaseURL is http://x.x.x.x:4433/public/
Retrieving munkireport scripts
Configuring munkireport

Installing managedinstalls
Installing munkireport

Uninstalling applications
Uninstalling appusage
Uninstalling ard
Uninstalling backup2go
Uninstalling bluetooth
Uninstalling caching
Uninstalling certificate
Uninstalling crashplan
Uninstalling deploystudio
Uninstalling detectx
Uninstalling directory_service
Uninstalling disk_report
Uninstalling displays_info
Uninstalling extensions
Uninstalling fan_temps
Uninstalling filevault_status
Uninstalling findmymac
Uninstalling fonts
Uninstalling gpu
Uninstalling gsx
Uninstalling homebrew
Uninstalling homebrew_info
Uninstalling localadmin
Uninstalling location
Uninstalling mbbr_status
Uninstalling munki_facts
Uninstalling munkiinfo
Uninstalling network
Uninstalling network_shares
Uninstalling power
Uninstalling printer
Uninstalling profile
Uninstalling sccm_status
Uninstalling security
Uninstalling servermetrics
Uninstalling service
Uninstalling smart_stats
Uninstalling softwareupdate
Uninstalling timemachine
Uninstalling usage_stats
Uninstalling usb
Uninstalling user_sessions
Uninstalling warranty
Uninstalling wifi
Preparing /Library/Preferences/MunkiReport
Setting preferences
Installation of MunkiReport v3.0.1 complete.
Running the preflight script for initialization
Traceback (most recent call last):
File "/usr/local/munki/preflight", line 6, in
from munkilib import reportcommon
ImportError: No module named munkilib
CHMM115523:~ sysadmin$

we searched extencively for solution and failed. finally approaching here for some clues and guidance to proceed further

Do you have Munki installed on that machine?


The demo folder does not exist

The demo folder does not exist, could you please provide demo midi file for model inference?

Thanks, it works really well.


Fixed #41 and #36: WebSocket+Indigo examples

This is based on the main branch. If PR #39 is merged first then the Indigo example will break - which is fine - will just need some minor changes.


Store other program metadata in a Program class

Sketch:
class Program:
    def __init__(self, text_program):
        """Converts pioasm text to encoded instruction bytes"""
        ... # all of old 'assemble()' is here
        self.assembled = array.array("H", assembled)
        self.sideset_count = sideset_count
        self.sideset_enable = sideset_enable
        # automatic loop target values, etc

    def make_state_machine(self, ...):
        import rp2pio # defer import so module can be used on std python
        return rp2pio.StateMachine(..., sideset_enable=self.sideset_enable, ...)

def assemble(text_program):
    return Program(text_program).assembled

This could also help when adding support for wrap / wrap_target, the values would be members of Program and passed to StateMachine via Program.make_state_machine.

The other thing I was considering was a function that produces the appropriate kwargs.
def get_kwargs(pio_source):
    ...

sm = rp2pio.StateMachine(pin=board.FOO, **get_kwargs(src))


API to extract unencoded string data

It would be nice to have a API in jsmn that takes a string token and returns the unescaped version of that string (possibly in UTF-8 for \u escapes).

Already discussed in at least #48


fix test and move test sessions to nox

I realized the test were not robust enough to continue working on my set-up, some issues related to the isolation of environments.
I'm not a big fan of the tox syntax and I prefer to stick to the pure python nox sessions that essentially do the same thing.
We need to:

[ ] create some default session (lint, mypy etc..)
[ ] create a test session
[ ] make sure the test session run succesfully locally

I think this is closed and released. If there is anything else to do, just mention it and reopen :)


[SSCP] Report JIT compiler errors from CUDA

Use CU_JIT_ERROR_LOG_BUFFER when JIT compiling CUDA code, and report the error log in case of compilation failure.
For example, the following kernel I might (did) naively expect to work
using Allocator = sycl::usm_allocator<char, sycl::usm::alloc::shared>;
std::vector<char, Allocator> vals(Allocator{q});
static constexpr std::size_t buff_len = 20;
vals.resize(buff_len);

sycl::range<1> work_items{vals.size()};
q.submit(
	[&](sycl::handler & cgh)
	{
		cgh.parallel_for<class vector_double>(
			work_items,
			[buff = vals.data()](sycl::id<1> tid) {
				std::format_to_n(
					buff, buff_len, "Hello from thread {}", static_cast<int>(tid));
			});
	});
q.wait_and_throw();

Without this patch the output is not too helpful
1: [AdaptiveCpp Error] from /home/dave/workspace/conan_recipe_staging/adaptivecpp/all/src/src/runtime/cuda/cuda_code_object.cpp:96 @ build_cuda_module_from_ptx(): cuda_executable_object: could not load module (error code = CU:218)
1: [AdaptiveCpp Error] from /home/dave/workspace/conan_recipe_staging/adaptivecpp/all/src/src/runtime/cuda/cuda_queue.cpp:692 @ submit_sscp_kernel_from_code_object(): cuda_queue: Code object construction failed

With this patch the output tells you why it failed
1: [AdaptiveCpp Error] from /home/dave/workspace/conan_recipe_staging/adaptivecpp/all/src/src/runtime/cuda/cuda_code_object.cpp:112 @ build_cuda_module_from_ptx(): ptxas fatal   : Unresolved extern function 'memchr' (error code = CU:218)
1: [AdaptiveCpp Error] from /home/dave/workspace/conan_recipe_staging/adaptivecpp/all/src/src/runtime/cuda/cuda_queue.cpp:692 @ submit_sscp_kernel_from_code_object(): cuda_queue: Code object construction failed

I had a look at the unit tests and couldn't see anywhere a test for CUDA-specific failure modes could fit. Happy to add one if someone can point the way.

Thanks! Though the intention of the fixup! commits was that they be squashed in before merge ;)


Some Recipes not working

Issue Description:
Some of my recipes are not working - This is on SpongeForge Server on a modpack I created for my public server.
What happens:
After added recipes to scripts, some recipes do work and others don't - those which don't work don't throw any errors, they just don't come up in JEI and are uncraftable. There appears to be no correlation.
What you expected to happen:
All my recipes to work, obviously.
Scripts:
With recipes which doesn't work:
https://pastebin.com/k5BeWwV2
https://pastebin.com/MR8CyUET
https://pastebin.com/qqF6Kd4E
With recipes which do work:
https://pastebin.com/c7fpbLph
https://pastebin.com/jqBtMzWv
https://pastebin.com/paVHHvZn
Minetweaker.log file:
https://pastebin.com/P8L1VWT3

Affected Versions (Do not use "latest"):

Minecraft: 1.10.2
Forge: 2254
Crafttweaker: 3.0.24
SpongeForge: 2259
Mods: https://pastebin.com/S3W9B77p

Your most recent log file where the issue was present:
My latest log file for the server is over 200k lines long so I believe I've pasted the relevant info:
https://pastebin.com/N8zReAQs
Knowing my luck, this is something stupid I've missed, any help is appreciated.

Alice
DragonTechMC

Can this be reproduced on a normal forge server?

https://pastebin.com/k5BeWwV2 should be: (forgot to close the first emerald ore dictionary ingredient)
recipes.remove(<storagedrawers:upgradeStorage:4>);
recipes.addShaped(<storagedrawers:upgradeStorage:4>, [
 [<ore:gemEmerald>, <ore:stickWood>,     <ore:gemEmerald>],
 [<ore:stickWood>, <storagedrawers:upgradeStorage:3>, <ore:stickWood>],
 [<ore:gemEmerald>, <ore:stickWood>,     <ore:gemEmerald>]]);

https://pastebin.com/MR8CyUET should be: (Forgot to close the first ore dictionary for stone)
recipes.remove(<progressiveautomation:PlanterStone>);
recipes.addShaped(<progressiveautomation:PlanterStone>, [
  [<ore:stone>, <ore:stone>, <ore:stone>],
  [<ore:stone>, <progressiveautomation:Planter>, <ore:stone>],
  [<ore:stone>, <ore:bricksStone>, <ore:stone>]]);

https://pastebin.com/paVHHvZn should be: (Ender pearl ore dictionary has an underscore)
recipes.remove(<enderstorage:enderStorage>);
recipes.addShaped(<enderstorage:enderStorage>, [
  [<ore:rodBlaze>, <ore:blockWool>, <ore:rodBlaze>],
  [<ore:obsidian>, <ore:chestWood>, <ore:obsidian>],
  [<ore:rodBlaze>, <ore:ender_pearl>, <ore:rodBlaze>]]);

@bookerthegeek Thanks for pointing the missing >'s - I didn't think to look for those kinds of mistakes because it should have errored for that, no? The Progressive Automation and Storage Drawers ones now work fine.
Also ore:enderpearl is correct
https://gyazo.com/8a09da9e7e14ad4c6258899d7145b1cc
And again, if it was incorrect, it should have errored?
There's nothing wrong with that EnderStorage script.

Edited to reflect this. I have no idea as to why that does not work.
@yepidoodles Can you place the enderchest in the world, dye all the colors a different color, then what is the output for MT hand?

Don't see how it helps but
<enderstorage:enderStorage>.withTag({Frequency: {middle: 3, left: 2, right: 4}})

As on my 1st server it works fine, I made a list of mods which are different between the servers (They have all the same plugins)
Mods 2nd server has but 1st doesn't:

Agricraft
Chameleon
Chisel
Cooking for Blockheads
Decocraft
Draws and Bits
Earth Works
Ender Zoo
Flat Coloured Blocks
Hunger Overhaul
Infinity Lib
Pam's Harvestcraft
Progressive Automation
Roguelike dungeons
Ruins
Spice of Life
Storage Drawers
Super Circuit Maker

Any chance any of these mods are interfering?

Try This. If it does not work create a chest with all white and use that as the output stack
recipes.remove(<enderstorage:enderStorage>);
recipes.addShaped(<enderstorage:enderStorage>.withTag({Frequency: {middle: 0, left: 0, right: 0}}), [
	[<ore:rodBlaze>,<ore:blockWool>,<ore:rodBlaze>],
	[<ore:obsidian>,<ore:chestWood>,<ore:obsidian>],
	[<ore:rodBlaze>,<ore:enderpearl>,<ore:rodBlaze>]]);

Default all white ender chests are just <enderstorage:enderStorage> - I did try with the tags but that also didn't work.

Link for the pack please?

https://www.technicpack.net/modpack/elbeta.705651

I am (was, see "solution") also having this issue. I'm trying to remove the default recipes and add my own for Mekanism's steel/obsidian armor. No errors are being reported and only the two chestplate do not work. I'm not using Sponge and I've tried 6 different Forge versions from 2815 (rec) to 2297 (current newest). All other scripts I have function correctly (BWM, MPS and IE).
In 0remove.zs:
# Mekanism
recipes.remove(<mekanismtools:SteelHelmet>);
recipes.remove(<mekanismtools:ObsidianHelmet>);
#recipes.remove(<mekanismtools:SteelChestplate>); # Currently disabled removing vanilla or item is uncraftable
#recipes.remove(<mekanismtools:ObsidianChestplate>); # See above note
recipes.remove(<mekanismtools:SteelLeggings>);
recipes.remove(<mekanismtools:ObsidianLeggings>);
recipes.remove(<mekanismtools:SteelBoots>);
recipes.remove(<mekanismtools:ObsidianBoots>);

mek.zs:
# Steel armor
recipes.addShaped(<mekanismtools:SteelHelmet>,
  [[<ore:plateSteel>, <ore:plateSteel>, <ore:plateSteel>],
  [<ore:plateIron>, <minecraft:leather_helmet>, <ore:plateIron>]]);
recipes.addShaped(<mekanismtools:SteelBoots>,
  [[<ore:plateSteel>, null, <ore:plateSteel>],
  [<ore:plateIron>, <minecraft:leather_boots>, <ore:plateIron>]]);
recipes.addShaped(<mekanismtools:SteelChestplate>,
  [[<ore:plateSteel>, <minecraft:leather_chestplate>, <ore:plateSteel>],
  [<ore:plateSteel>, null, <ore:plateSteel>],
  [<ore:plateIron>, <ore:plateSteel>, <ore:plateIron>]]);
recipes.addShaped(<mekanismtools:SteelLeggings>,
  [[<ore:plateSteel>, <ore:plateSteel>, <ore:plateSteel>],
  [<ore:plateSteel>, <minecraft:leather_leggings>, <ore:plateSteel>],
  [<ore:plateIron>, null, <ore:plateIron>]]);

# Obsidian armor
recipes.addShaped(<mekanismtools:ObsidianHelmet>,
  [[<ore:itemCompressedObsidian>, <ore:itemCompressedObsidian>, <ore:itemCompressedObsidian>],
  [<ore:plateSteel>, <minecraft:leather_helmet>, <ore:plateSteel>]]);
recipes.addShaped(<mekanismtools:ObsidianBoots>,
  [[<ore:itemCompressedObsidian>, null, <ore:itemCompressedObsidian>],
  [<ore:plateSteel>, <minecraft:leather_boots>, <ore:plateSteel>]]);
recipes.addShaped(<mekanismtools:ObsidianChestplate>,
  [[<ore:itemCompressedObsidian>, <minecraft:leather_chestplate>, <ore:itemCompressedObsidian>],
  [<ore:itemCompressedObsidian>, <ore:itemCompressedObsidian>, <ore:itemCompressedObsidian>],
  [<ore:plateSteel>, <ore:plateSteel>, <ore:plateSteel>]]);

Solution: Delete the entire minecraft instance and create a new one. This seems to have allowed the scripts to work. Why? I honestly have no clue.

Delete the entire minecraft instance, you mean the server? The map?

No. The way MultiMC5 works is that you have separate minecraft "installations" that it keeps track of and manages. This way, you can have 3 1.10.2 instances ("installs") with different mod packs, one for 1.6.4, 2 for 1.11.2, etc.
What I did was save the script files and delete everything else, equivalent to the .minecraft folder on windows. Mind you, this is less than ideal as I know how have to sort through all the configs and update them again. I also didn't have any worlds aside from the flat testing so I didn't lose anything of value, aside from time.

Oh I assumed MultiMC5 was a Admin Panel, sorry. I'm glad this fix worked for you, but it certainly won't for me :/

Since I can't reproduce this, and an answer was never given as if this works without Spongeforge, I'm closing it.

And idk about a normal forge server, but it doesn't work in Single player

For what it's worth I haven't had this issue since and I'm betting it was most likely something I messed up.

And idk about a normal forge server, but it doesn't work in Single player

Well sponge forge can be installed client side, so it not working in single player means nothing unless the single player test was done without spongeforge.

As far as I am aware, SpongeForge can not be installed client side... It is a server software/ mod. So no, SpongeForge was not installed client side.

can I get a list of the client mods?
/mt mods
``` should do

As far as I am aware, SpongeForge can not be installed client side...

Please note I didn't do much testing as I have no real interest in running a "hacked together mess" and recreate the whole mcpc era nightmare.

@jaredlll08 The mod list was posted in the OP unless changes is at https://pastebin.com/S3W9B77p

That list doesn't look generated though, so a generated list would be
better, with mod versions etc
On Jul 20, 2017 4:02 AM, "BookerTheGeek"<EMAIL_ADDRESS>wrote:

@jaredlll08 https://github.com/jaredlll08 The mod list was posted in
the OP unless changes is at https://pastebin.com/S3W9B77p
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jaredlll08/CraftTweaker/issues/189#issuecomment-316574617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFS9xZv2osx_oFGlP4y10J-AsM5lFW4Hks5sPrU4gaJpZM4MzJ6-
.


wiki: new naming scheme for installation media

I'm aware that this is not the best place to report this, but I couldn't find a better one.
The Installation documentation still mentions the old naming scheme for installation media (memstick, LiveCD, serial), but starting with 15.1.6 a new scheme was introduced (vga, cdrom, serial).
I would've changed it, but I don't have write access to the Wiki. :-)
As a sidenote, I think the name 'vga' for the USB installer image is slightly misleading. On the other hand, it's hard to please everybody. ;-)

No problem and thanks for mentioning it, I personally really like the issue tracker of GitHub :-)
I have replaced the media types with the new ones.
If you want an update account on the wiki, just create an account and let me know your account name.

@AdSchellevis Thanks, and nice that you've changed the version and platform too :+1:
My wiki account name is Fraenki ;)

@fraenki what would be a better name for "vga"? I've tried to keep the names as small as possible, avoiding stuffing of "memstick-usb-serial-with-vga-on-the-side", "memstick-vga-but-no-serial", etc. I'm overstating for emphasis, but I'm not against tweaking the names further given there are better ideas.  I've changed your wiki account, you should be able to edit now.

@fichtner Hm, I'd rather vote for a more expressive naming scheme, but I can't think of anything without going back to long filenames. I think I can get used to the new scheme, too. Don't worry.
@AdSchellevis Edit works, thanks!

@fraenki expressive vs. explicit -- I do think the names are expressive, but not explicit and you probably mean explicit? I like one word names, even though they don't reflect the full scope, they are easier to remember and communicate. If we can find better words for the images we should! :)


Document Tag versioning strategy

Changes
Mention the Tag versioning strategy
Checklist

[x] Read the contribution guide and accept the [code](../blob/main/  CODE_OF_CONDUCT.md) of conduct
[x] Readme (updated or not needed)
[x] Tests (added, updated or not needed)

Note that this used to be documented, but then I removed it because too many people were confusing it with the automatic versioning option, which we are recommending for most cases.
It solved the problem, at the cost of an unclarity.
Maybe we can come up with a good way to indicate the subtle difference and reduced complexity of "auto" when compared to "tag" option.

Maybe we can come up with a good way to indicate the subtle difference and reduced complexity of "auto" when compared to "tag" option.
Maybe we could do the same thing that is done for the "custom" option: Label it with (advanced users)
We could also add a line indicating why "semantic" is recommended over "tag".

Ah, that's interesting. In that case I think marking it as "advanced" makes sense.
As a side-note I am getting this warning from the build (even though I'm not attempting to build for Android right now):
 v0.0.98

Warning: Could not parse "0.0.98
" to semver, defaulting android version code to 1

Although that seems like it should be a separate issue. From looking at the code I believe that there is a trailing newline that should be removed before passing the value on.

I think "Advanced" or "this requires to manage versions manually, using other actions" would be helpful. 👍
The warning indeed looks to be a bug related to a newline.


Add installation instructions to the README

@RaoOfPhysics has already added a few installation instructions in #3, and I'm just running through them now to check they work (and to be able to run the MVP locally!)
I think there are probably a few things missing - do you need to have BeakerBrowser or Dat installed already?
It looks like I need npm too?
If anyone has time to comment here, I'm super happy to actually do the additions to the README (I just need help knowing what to do!)
Thank you ✨

Also happy to help! Just tag me. :)


2024-12-11_21-08-43_master-next_aws-cli-v2

Automatically created.

Backport failed for kirkstone-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin kirkstone-next
git worktree add -d .worktree/backport-10740-to-kirkstone-next origin/kirkstone-next
cd .worktree/backport-10740-to-kirkstone-next
git switch --create backport-10740-to-kirkstone-next
git cherry-pick -x 3fa1480c6ed20d76fdbd75ff812f9ce00f0e5181

Backport failed for scarthgap-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin scarthgap-next
git worktree add -d .worktree/backport-10740-to-scarthgap-next origin/scarthgap-next
cd .worktree/backport-10740-to-scarthgap-next
git switch --create backport-10740-to-scarthgap-next
git cherry-pick -x 3fa1480c6ed20d76fdbd75ff812f9ce00f0e5181

Successfully created backport PR for styhead-next:

#10745


From the docs i could figure out that it takes script, output and script_parts.
It would be a great help if someone can give an example of the command passed in the below function.
def match(command):
    if command.script_parts and '&&' not in command.script_parts and command.script_parts[0] == 'sudo':
        return False

    for pattern in patterns:
        if pattern in command.output.lower():
            return True
    return False

Thanks

Hi! :wave:
Here is the definition of the class command is an instance of:
https://github.com/nvbn/thefuck/blob/51e4e8728098903a87fb6d1cd2083460d439f357/thefuck/types.py#L13-L83


DRILL-8364: Add Support for OAuth Enabled File Systems

DRILL-8364: Add Support for OAuth Enabled File Systems
Description
This PR adds support for Drill to query file systems which use OAuth for authorization and authentication.  This PR also adds support for Drill to query Box.com.
Documentation
See README for Box for documentation.
Testing
Added unit tests and tested manually.

Would it be possible, for the sake of structuring the code base, to move the Box and Dropbox filesystem implementations out to contrib/storage-box and contrib/storage-dropbox modules (while keeping the current Java package organsiation)?

@jnturton
I don't know.  Unlike the regular storage plugins, which are pretty self-contained, to use a file system, you have to "register" it in the FileSystemPlugin class. (See below).   I assumed, perhaps incorrectly, that new file systems had to be in the java-exec package.  I'm not a packaging expert, so do you know what we'd have to do in order to move these to contrib?
https://github.com/apache/drill/blob/53e4227650755607db961b758b7966b1d6d4582f/exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/FileSystemPlugin.java#L93-L96

@jnturton I don't know. Unlike the regular storage plugins, which are pretty self-contained, to use a file system, you have to "register" it in the FileSystemPlugin class.

Okay, let's leave that refactoring for a separate PR.


Remove hard-coded path.

As I tried this code in an install space, I found many code paths started complaining modules isn't found or modelPath doesn't not exists. And I attributed those errors to the hard-coded path in current implementation.
This pull request is to remove them and use more generic way to discover the file paths for the dependencies.

cc @robotpilot

@seanyen
Thank you for your contribution to the library. I confirmed everything works fine with this PR so I will merge it.
Ryan


EtherscanConfig missing the optimismSepolia api url

Describe the bug
Hello,
I am using the etherscan plugin to get the read and write function from the contract from optimism sepolia testnet. But typescript keep throwing this warning
The expected type comes from property 'chainId' which is declared here on type 'EtherscanConfig<1 | 5 | 11155111 | 10 | 420 | 137 | 80001 | 42161 | 421613 | 56 | 97 | 128 | 256 | 250 | 4002 | 43114 | 43113 | 42220 | 44787>'
And If i run the wagmi generate command I get this error

After debugging the code I found that Etherscan API key is missing the optimism sepolia API key.

Thanks in advance.
Link to Minimal Reproducible Example
No response
Steps To Reproduce
import { defineConfig } from '@wagmi/cli';
import { actions, etherscan, react } from '@wagmi/cli/plugins';
import { optimismSepolia } from 'wagmi/chains';
import "dotenv/config";

export default defineConfig({
  out: 'src/generated.ts',
  contracts: [],
  plugins: [
    etherscan({
      apiKey: process.env.ETHERSCAN_API_KEY!,
      chainId: optimismSepolia.id,
      contracts: [
        {
          name: 'uSwap',
          address: {
            [optimismSepolia.id]: process.env.CONTRACT_ADDRESS as `0x${string}`,
          },
        },
      ],
    }),
    react(),
    actions({
      readContract: true,
    }),
  ],
});

Wagmi Version
1
Viem Version
1
TypeScript Version
^5.2.2
Check existing issues

[X] I checked there isn't already an issue for the bug I encountered.

Anything else?
No response

Feel free to open a PR adding this to @wagmi/cli@2


Pattern:  Segmented long-form design provided by Web Simplification

Notes

Lila is the web simplification designer.  This issue is to track the transition of the design work to the DDS engineering team members.

Additional Information

Designs
Design specs
Designs are scheduled to be available on Jan 17.  The review meetings with DDS should be scheduled early the following week.

Acceptance Criteria

[ ] Specs received from Web Simplification and reviewed with DDS engineering
[ ] Meetings conducted to review the designs

Meetings with Web Simplification team members scheduled Jan 20 & 21.