Accessing read-only paths

I am currently attempting to package this in Nix but upon running the program, it attempts to write to the read-only nix store /nix/store/258d5k9hcdqp79m6izfzh6936lmf3j1l-adgobye-4.0.2+v4IsBrokenPullMe/lib/adgobye/Plugins
Shouldn't plugin data be managed in the users .config or .local directory?

Err, yes, AdGoBye expects to have all its files in the same folder as the binary. If I made it a first class Linux app, that probably should go into .local and config instead.
I'm not sure about the filesystem restrictions of NixOS, else I'd be fine with doing a patch to allow picking the working directory for the app.

It's pretty much packages are built and put into the nix store (/nix/store) which is read-only so any binary that does things based on the location of the binary will probably have problems. There are probably ways to fix this on the Nix side but I think this is an appropriate upstream issue anyways

Yeah no, completely valid, I'll take a look at it when I find the time.

Do you mind testing the artifact for https://github.com/AdGoBye/AdGoBye/actions/runs/11190863213? I did this fairly lazily (I'm tired and not really into it today) but it should be good enough if I understand correctly.
Make sure there's no appsettings.json next to the binary, and either start it once and let it create $XDG_DATA_HOME/AdGoBye for you or create it yourself and move appsettings.json into the folder.

I'll give it a test tomorrow

I'm actually building it from source because it's a little easier than fetching a binary.
When building, it fails with these errors:
/build/adgobye/AdGoBye/Program.cs(32,32): error CS0117: 'Log' does not contain a definition for 'Log' [/build/adgobye/AdGoBye/AdGoBye.csproj]
/build/adgobye/AdGoBye/PluginLoader.cs(105,23): error CS0117: 'Settings' does not contain a definition for 'Options' [/build/adgobye/AdGoBye/AdGoBye.csproj]
/build/adgobye/AdGoBye/PluginLoader.cs(110,17): error CS0122: 'Logger' is inaccessible due to its protection level [/build/adgobye/AdGoBye/AdGoBye.csproj]

v4.0.2 builds fine

My bad, I think these were usually obscured by the debug macros, I'm honestly not sure how I didn't catch that sooner.
9ef438f4bffdc032ea7dc7d1fa7c01f5527ce21b should fix that

It appears to run now! I haven't tested its functionality but I probably will soon. Could the appsettings.json be created if it doesn't already exist? That's all I can think of

Yeah, I think we can embed appsettings.json as a embedded Resource and use that for bootstrapping, we get access to resources per Assembly.GetManifestResourceStream.
I'll look into it when I feel a bit more like touching AGB again, this was definitely just the lazy unbreak-it-now implementation for now.


Bug: unsupported ts-node-dev package in CLI command approach

Versions

Node:
OS:

Reproduction

Somehow CLI command approach overwrites "src" -> "dist" in paths set by ENV variable

Steps to reproduce

Install the project
Set "TYPEORM_SEEDING_SEEDS" to "src/data/seeds/*.ts"
Run the CLI with "ts-node-dev"

What is Expected?
It should work
What is actually happening?
Seeder files are not found

@jzfre yes, this is the default behavior. The library tries to resolve the outDir in the tsconfig.json to allow execution in transpiled state as well as with ts-node.


Super confusing docs on how to simply add mermaid theme into the presets for 'classic'

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
One of the most frustrating parts in the docs is when you install and get Docusaurus running, but cannot find how to properly configure 2 or more themes.  The examples always use module.exports = { } but yet the default shipping config of Docusaurus uses an object with simply config.  So when someone wants to simply add let's say Mermaid theme for their markdown pages, it's super confusing how to simply add Mermaid as a theme into the docs, or blog sections.
I would love to improve the docs, but need the answer before I can contribute.
I wanted to do something as easy as this:
module.exports = {
  config,
  themes: ['@docusaurus/theme-mermaid'],
}

but of course that syntax isn't respected.
I did add the other config parts into the preset-classic ThemeConfig section
      // MERMAID CONFIG //
      markdown: {
        mermaid: true,
      },

      mermaid: {
        options: {
          maxTextSize: 50,
        }
      }
      // END OF MERMAID CONFIG //

but of course the pages do not show mermaid styling, but only the raw codeblock text of the mermaid graph.
I suspected I needed the mermaid theme still to be added, but on the preset-classic, not sure where or how.
In general, how to use the default installation to add or use 2 themes.  Because module.exports = config is not easily understood other than it's a Config type.
Self-service

[x] I'd be willing to address this documentation request myself.

Maybe we can try to improve here, but we can't really spare teaching the basics of Node and JS.
This:
const config = {...};

module.exports = config;

Is equivalent to:
module.exports = {...};

Therefore all you should care is what comes between the curly braces. The docs tell you to add two fields called markdown and thèmes (neither of which you should have initializing the new project).

@Josh-Cena I got it working, and it was something more simple ... I removed the config constant and only wrapped with module.exports = { } as you were trying to explain.  For some reason, I thought making the change below would break config again, which I think I tried but had a red herring error message because of wrongly placed mermaid config.  Anyways, got it working as you were trying to lead me into that basic direction.  So then a question arrives...
So I'm curious if all the examples and documentation say module.exports =  then why confuse initial users by having the first default config to even have this:
/** @type {import('@docusaurus/types').Config} */
const config = {
  title: 'Welcome to ...',
  tagline: 'A low-code API development platform',
  favicon: 'img/favicon.ico',
  etc...

rather than having this ? :
/** @type {import('@docusaurus/types').Config} */
module.exports = {
  title: 'Welcome to HOMI',
  tagline: 'A low-code API development platform',
  favicon: 'img/favicon.ico',
  etc...

I used the const config constant because it was needed for my IDE and TS to understand the line just above: /** @type {import('@docusaurus/types').Config} */
This line permits to tell your IDE the shape of the config object, provides autocompletion features and TypeScript typechecking to ensure you only use valid config.
See for example:

There's probably a way to make it work with module.exports, but couldn't find one, so if a JSDoc expert has any suggestion to make please let me know ;)
The intermediate config variable simply looked like a good-enough workaround

Note in the future, with the need to use ES modules in config, more and more sites will probably need to move to another form of config declaration using an async function, like this one:
/** @returns {Promise<import('@docusaurus/types').Config>} */
module.exports = async function createConfigAsync() {
  return {
    // your config here
  };
};

TLDR: there's not a single way/syntax to declare your config. You have the full power of Node.js, as long as you return an object that matches the shape we expect (or a function that creates that object).

OMG so helpful @slorber  you solved the puzzle basically.  And yes, I wanted to keep the autocompletion and TypeScript checking working and grieved that I lost it!
So then for this issue let's do a few things to improve easily! :

[ ] Add a // comment in the shipping docusaurus.config.js at the top such as:

// NOTE:  We use 'const config =' as simply an easy way to help give TypeScript hints for your IDE to work with this config file.  There is not a single way/syntax to declare your config within it, but note that it gets used much much further down this file, where, 'module.exports =' is the syntax that expects an object (or a function that creates that object, your choice) that returns the expected shape of a docusaurus-config See also: https://docusaurus.io/docs/api/docusaurus-config#overview

[ ] Update the https://docusaurus.io/docs/configuration page to state the above, preferably in a new section called '#about-the-default-docusaurus.config.js' that is placed last after '#custom-configurations', which will:

[ ] Also add @slorber 's image shown above to show that either syntax works, but only one syntax shows an error. Super F'ing helpful!
[ ] Also add @slober 's 'async' example into the page under '#custom-configurations'

Add a // comment in the shipping docusaurus.config.js at the top such as:

If we add a comment in the init template, I'd like the comment to be very light (2 lines max) and link to the doc page for more details.

We have 2 pages:

docusaurus.io/docs/api/docusaurus-config#overview: ref doc
docusaurus.io/docs/configuration: guide doc

IMHO:

Both pages should link to each other at the top in a very visible way.
Mentioning equivalent docs format (including the const config thing is more suited to live in the guide page
Agree that the async config section in a summary/details is not visible enough. We should add these in expended form to the guide page, but still mention it's possible to use async in the API ref page, and add another link to the guide page.

In general, if you want to add long/verbose documentation on some subject, I prefer it to be in the guides part

@slorber Looks like your normal convention of API referencing is with an blue INFO admonition typically at the top.  I saw this across about 5+ pages throughout the site.

Dev Preview of Configuration after update:

Dev Preview of API after update:


Display transfer errors: also on target

These error messages are displayed on mobile, and should also be displayed on target browser, when possible, if QR has already been scanned :

"This resource exceeds the 20MB limit"  (in fact this should happen before scan, so no)
"Error during transfer"

Android sends notif to backend: done.
Backend forwards notif to target client: done.
TODO target client handles error message (displays it to user).
TODO iOS sends notif to backend.

Related to (but distinct from) #235

Android sends notif to backend: done.
Backend forwards notif to target client: done.
Target client displays error message to user: done.
TODO iOS sends notif to backend.

Currently on iOS, we open camera only after successful upload.
When /new-gcs-urls or upload PUT fail, we don't have any qrKey no notify to yet.
We may try to catch error on /dispatch, but it is not exactly a file transfer error, and if /dispatch fails then it is likely that the error notification request would also fail.
Bottom line: target browser won't display upload error from iOS, because at that time the target is unknown yet.


Error When try to create an answer

When i receive an incoming call and try to call accept
i got error e = "Unable to RTCPeerConnection::createAnswer: Error (null)"
This happens whey i try to do as you in example , like the following :
 var answer = await sip?.createAnswer();
 await sip?.accept(sessionDescription: answer);

Any what what could be the problem of how i can solve it ?

I have java script client that works fine with my sip configuration and outbound is working fine with this client , only answering the incoming call is the problem

Any update ?

Please try using 2.2.14 and tell me if that works for you in the meantime i will fix it

can you try latest commit if it works for you and let me know?

I still need time to investigate it, thank you for your patience

Any update here ?

hi @mazen930 so i tried on two ios simulators with test sip credentials and everything seems to be working just fine i will need more info on your issue to reproduce the problem

Below is the full error appears when i try to create an answer , i use actual phone to test ,not an emulator , i appreciate if you can look it it
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Unable to RTCPeerConnection::createAnswer: Error (null)
#0      RTCPeerConnectionNative.createAnswer (package:flutter_webrtc/src/native/rtc_peerconnection_impl.dart:355:7)
<asynchronous suspension>
#1      JanusPlugin.createAnswer (package:janus_client/janus_plugin.dart:586:38)
<asynchronous suspension>
#2      Janus.acceptCall (package:maqsam/models/services/web_rtc_clients/janus_client.dart:177:18)
<asynchronous suspension>

After debugging , it appears that await sip?.handleRemoteJsep(event.jsep); was not configured properly when receiving incoming event , the error was ambiguous and  miss leading me in another direction , i suggest edit your example take that line of of the method that  handles the UI showIncomingDialogBox or something like that , anyway thank you @shivanshtalwar0  for your effort and help

Thanks for your debugging as well, i would appreciate if you could make a little pr for the same if you have time otherwise no problem i will do later

I added a pull request with changes needed to make the sip plugin work correctly , and i will also add another pull request to update the example to be more informative


Reference docs incorrect v1.18 kubectl run command

This is a Bug Report
Problem:
The reference documentation for kubectl v1.18 does not match the implementation regarding run command and its options --restart  and --replicas.
Proposed Solution:
Update the reference docs.
Pages to Update:

https://kubernetes.io/docs/reference/generated/kubectl/kubectl-commands#run, the sections for options replicas and restart are incorrect. The --replicas option is depreciated and run always creates a pod regardless of the --restart value.

https://kubernetes.io/docs/reference/kubectl/#operations, the following line:

kubectl run NAME –image=image [–env=“key=value”] [–port=port] [–replicas=replicas] [–dry-run=server

The --replicas option is depreciated.

/assign

Thanks for contributing @DragD


Update azuredeploy.parameters.json

remove parameters that Azure Stack can not consume
Change values to std values
Change schema path to correct path
** remove azuredeploy.azurestack.parameters.json file and use this file. Standardizes JSON templates and is easily consumable by Visual Studio

#sign-off

@KathMcB Merge conflict


fix(Checklist): update checklist donut chart colors

Contributes to https://github.ibm.com/security/novice-to-pro-components/issues/193
Minor design update
What did you change?
Just the colors for the Checklist donut colors, light and dark themes.
How did you test and verify your work?

[x] Storybook
[x] Signed off by designers Bridget Ndege and Cameron Calder

@elycheea The designers have signed off on this PR.


525 Response when Uploading Maps

Receive a 525 response "Origin SSL Handshake Error" when uploading from tmp/appmap/rspec.
No such error occurs when uploading from tmp/appmap/cucumber which occurs directly after.
See here for travis logs

This appears to be caused by problems on app.land: uploading the appmaps in tmp/appmap/rspec provokes "Memory quota exceeded" errors on the server.


[CherryPick] Fixed MddBootstrapInitialize scanning for DDLMs erroneously excluded packages with release != *.0.0

…packages with release != *.0.0 (#2248)
Cherrpick of #2248

Fix MddBootstrapInitialize filter erroneously excluding packages with release != *.0.0. Added logging to DDLM enumeration to improve troubleshooting failures

Incorporated feedback

/azp run

/azp run


⚠ GitLab Server has degraded performance

In a686d7a, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 9137 ms

Resolved: GitLab Server performance has improved in 67e8ee6.


Deleting a column (sometimes) breaks the table

I imported this table to my account (shapefile).
What I did:

Georeference by city names without using country
Delete cartodb_georef_status column
Georeference by city names by using "UK" as country

Noticed there was a column with countries, decided to try it

Delete cartodb_geo column which was the previous cartodb_georef_status column from the original table but trimmed because I exported/imported the shapefile version.
Crash. The table contents weren't being shown and the console included a 404, which was asking for the old column that doesn't exist now.

I haven't been able to get this last trace but this happened to me 4 days ago here
{
error: [
"column "nom" does not exist"
]
}

The table was this one. The second table affected was this one.
I'm not able to reproduce the issue but this has already happened to me ~3 times in the last week.
PD: No events in rollbar for the issue (or at least I wasn't able to find them related with my username).

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


fix int32 wire type

I think the WireType should be updated with options like go-amino: https://github.com/tendermint/go-amino/blob/master/reflect.go#L195

@TanNgocDo #24 I want to assign this issue to myself. But it seemed that I have no permission(?). I'm not familiar. Could you please check permission(?) or role(?) options in this repo?


full cover DataAnalysisServiceImpl

What is the purpose of the pull request
#3557
Brief change log

full cover DataAnalysisServiceImpl
simplify countQueueState method
delete try catch that will not throw any exeception
make up for the state of oblivion

Verify this pull request

I will fix this later

e2e failed, please rebase dev branch


Require Node v18 in Wrangler v4

Bump minimum supported Node version to Node v18 in Wrangler v4

Tests

[ ] TODO (before merge)
[x] Tests included
[ ] Tests not necessary because:

E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)

[ ] I don't know
[x] Required
[ ] Not required because:

Public documentation

[ ] TODO (before merge)
[ ] Cloudflare docs PR(s):
[x] Documentation not necessary because: will be part of v4 docs

also there's a line in the c3 nuxt templates that might need to be updated?
writeFile("./.node-version", "17");

also there's a line in the c3 nuxt templates that might need to be updated? writeFile("./.node-version", "17");

I think we can fix this without a major, so I'll open a separate PR


473 regular backups

Related to #473

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

62 of 62 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 95.066%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6559

Relevant Lines:
6672

💛  - Coveralls


CLOUD-665 added new default blueprint

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1056/


refactor: migrate demos/default to package path

Description

Documentation

Tests

You can test this change yourself like so:

TODO

Relevant links (GitHub issues, etc.) or a picture of cute animal

Superseded by v5.


Optimizing MCSS atom mapping

Since the MCSS atom mapping is so critical to success of the geometry proposal, we should optimize this at some point.
There are many different modifiers that can be used to fine-tune the atom and bond matching. We could even go further and use set string or integer atom and bond types that are used in the matching, which could allow us to match based on forcefield parameters---as I did in mmtools, though there are issues with how GAFF treats aromatic atom types---or create our own properties based on, say, the ring size (e.g. assign a 6 to atoms in 6-membered rings, or 5 to atoms in 5-membered rings).
There is a recipe in the Python OEChem cookbook for depicting substructure matches that might help us fine-tune these by looking at all kinase inhibitor matches and amino acid matches.
This isn't critical right now, but something that will ultimately be responsible for the success of this method.

This visualization recipe is also pretty cool.

We might also consider the MCSS scheme used in LOMAP2, though it seems to use RDKit rather than OpenEye tools.

I think we've sort of come to the conclusion that there is no one single atom scheme that works for all transformations. I think the AtomMapper and associated example is a good compromise to allow for easy testing of these things


Rigidbody velocity get set

Motivation and Context
We can currently manipulate objects kinematically via position|orientation and dynamically via force|torque. This change sets the stage for velocity control by adding support for getting|setting rigid body linear|angular velocity via Bullet physics.
Viewer demo updates: (for demo purposes: happy to modify/remove for final merge)

Added velocity control 'E' and anti-gravity mode 'G' for most recently added object with on-screen text to display current setting.
Added commandVelocity variable to store and apply a velocity control setting.

Notes:

This enables setting initial velocity for a sim step, not constant velocity (forces will change the velocity within the simulation step). Therefore use shorter steps and remove/counter forces (such as gravity) for more accurate control.
This does not allow setting velocity for objects with MotionType::KINEMATIC or MotionType::STATIC. In other words, it only currently enables velocity manipulation for MotionType::DYNAMIC BulletRigidObjects.

How Has This Been Tested
New C++ test and viewer demo:
Example 1: anti-grav velocity control of cheezit box

Example 2: velocity control with gravity of chef_can

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

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

See my inline comments. Thanks!

Thanks for the comments. The viewer changes were really just to demonstrate the functionality in the videos and I've removed them until the full velocity control feature set is ready.


Does not work in rails 4 because of activesupport 5 req...

Not sure if we would like to enable support for old rails version.

Okay that's fine just throw it in the readme

Added info to readme in commit fd2af72b18a2ba5013df06e6ca1d6232450e1a24


How to Change overall font

How to change font family

I also tried to change it but the current Theme Provider does not have the option to specify a font.

That's correct. As of right now there is no way to the font. Hope this helps @novfal .

Ok thanks


Regarding Smart Contract Upgradeability using SolidityJS Tool

Hello Solana Team,
Is it possible to implement smart contract upgradeability similar to Ethereum using Solana-Solidity-JS toolkit.

It's possible to upgrade solana BPF programs:
https://edge.docs.solana.com/cli/deploy-a-program#redeploy-a-program
We need an example of this in the JS library.


Add grid order arg fix column major calculation

Add GridOrder to the API to control the order that items are shown in

RowMajor -> left to right priority
ColumnMajor -> top to bottom priority

Tweaked the demo for more flexibility. Instead of adding new grid sections to show a different number of items, instead add a slider that will dynamically change how many items are displayed in the grid. Useful for testing and debugging various different scenarios with different icon counts and window sizes.

Is this taking my existing hidden implementation (bool: columnFirst) and productionifying it?
The diff was kinds hard to follow due to reordering, but I think the main point of difference is lines 125-152.
Can you explain a bit why its needed over what the original implementation was doing?


Publish 'pre' packages

PyPi has support for pre-release packages. The current version of Falcon (0.3.0-pre1?) has a few features that I'd like to use (like error messages for 404 status codes), but I'd like to install using pip. Would it be possible to publish a pre-release of 0.3.0?

The required changes are actually in 0.2.0. Clearly, I haven't had enough coffee this morning.


🎁: Handover - Exclamation mark

Expected outcome
It is clear to the user what package the exclamation mark belongs to
Today users are not sure if this belongs to the left or right package:

Business Value / Developer Experience
Prevent user from reading the data wrong
Acceptance Criteria

[x] It is clear what package the exclamation/warning belongs to

https://github.com/equinor/cc-components/pull/683


Rotten village fence

The plugin seems to try to path through the rotten village path as if it doesn't block the path in the tai bwo wannai area.

This issue was fixed in 9a9667c29fcdf437d9b2ddd1e0dde498ec7eb89c and will be part of the next plugin update.


Siddarth

i have made changes and added myself please go through it and if any issue occurs let me know. if know then please merge my pull request.

The json.txt file still exists. Remove that and make sure you commit only the JSON file

Alright thanks.
On Sun, 27 Sep 2020, 13:12 Chandu<EMAIL_ADDRESS>wrote:

The json.txt file still exists. Remove that and make sure you commit only
the JSON file
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenSourceTogether/Hacktoberfest-2020/pull/112#issuecomment-699599085,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUBFHPV3QB7JXJNKYRBWIDSH3UFBANCNFSM4R3L2EBQ
.


Start MockLogAppender before adding to static context

I observed a test failure with the message
'Attempted to append to non-started appender mock' from an assertion in
OpenSearchTestCase::after. I believe this indicates that a
MockLogAppender (which is named "mock") was added as an appender to the
static logging context and some other test in the same JVM happened to
cause a logging statement to hit that appender and cause an error, which
then caused an unrelated test to fail (because they share static state
with the logger). Almost all usages of MockLogAppender start it
immediately after creation. I found a few that did not and fixed those.
I also made a static helper in MockLogAppender to start it upon
creation.
Signed-off-by: Andrew Ross<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check

Let's backport to 1.x

Does this still need to be backported?

Backport PR here: https://github.com/opensearch-project/OpenSearch/pull/1659


Banner ads overlapping the page

The banner ads are displayed over the app content.

Hi @merbin2012,
Could you please explain your problem in more detail and share a sample code to reproduce your error?
Thank you for your interest.

I didn't not expected huwai provide this much worst experience, see the date of issue opened, and your reply. Any more I'll remove all app from huwai store and leave huwai ads. 🤮

Hi @merbin2012 , as @sinemsevilmis  said we couldn't reproduce your issue, we can only help you if you provide a detailed explanation of your problem and a minimum reproducible project or sample code if possible. Since we didn't get any response from you over the past month we have closed the ticket yet if your problem still exist you can always re-open the issue.

So is it good after 3months reply. Solving issues are second, first you have to give proper response soon as possible.

Huawei's official HMS Cordova plugins are wrappers of native HMS kits, thus when we encounter an issue on one of our kits we first try our native demo applications, Cordova demo application and other platform kits such as Flutter or React Native. The problem solving process of bugs may take some time regarding the provided very limited information from your side and the running process of updates and other plugin maintenances, sorry for the inconvenience.
I'm re-opening the issue, for further investigation please provide a detailed explanation of your problem and a minimal reproducible sample code or project if possible.

Ok
On Wed, 11 May, 2022, 2:14 PM Serdar Can, @.***> wrote:

Reopened #74 https://github.com/HMS-Core/hms-cordova-plugin/issues/74.
—
Reply to this email directly, view it on GitHub
https://github.com/HMS-Core/hms-cordova-plugin/issues/74#event-6588122292,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4EGEACZ27V63QICWK4PQLVJNXNXANCNFSM5MLZMHGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


libspdlog.so.1.13 needed installed version of waybar = 0.11.0-4

Hi there!
Every time I try to run Waybar I get the error that I need spdlog 1.13, whilst I have 1.15 installed and Waybar wouldn't need 1.13
So why doesn't it want to run?
I tried to uninstall Waybar and do yay -S spdlog-1.13.0-1-x86_64.pkg.tar.zst but I get an error that:
warning: downgrading package spdlog (1.15.0-1 => 1.13.0-1)
resolving dependencies...
looking for conflicting packages...
error: failed to prepare transaction (could not satisfy dependencies)
:: installing spdlog (1.13.0-1) breaks dependency 'libspdlog.so=1.15-64' required by waybar
I dont know what to do anymore, installed waybar and removed it a couple of times to, or tried to download the waybar-git. But they all give the same message.

the reason is, that during the PGKBUILD of waybar it pins the current version of libspdlog.so.
My solution was to download the package (PKGBUILD) and change the dependency from "libspdlog.so to" spdlog and then building and installing it (see my pkgbuild testing mirror here line 21). This change causes the package to depend on any version of spdlog (at least on my machine :smile:).
Another solution in the AUR is (it is waybar-git but the same error):
https://aur.archlinux.org/packages/waybar-git#comment-998777

what is your waybar version?

your previous comment worked too! thanks for the help @TripleTrable ! I changed the line in PKGBUILD

got it, thank you for your help and explanation! works good now


PATRIC global search issue

When I searched for "MDR" in global search field, it returned 165 genomes. But when I clicked on the 165 genome link, the result page said "Showing 33252 genome(s) for: MDR"

This was due to session issue. Fixed and available on beta.


[QUERY] Azure.Storage.DataMovement upload from URL

Library name and version
Azure.Storage.DataMovement v12 beta2
Query/Question
We need to upload a few million images to a azure storage using Azure.Storage.DataMovement. Our files all come from MLS's and we are not allowed to use the image urls on our site, so we have to host them ourselves. Does Azure.Storage.DataMovement have the ability to upload files in bulk from a list of URLs? What is the best approach for getting these files into our storage account?
Environment
No response

@johnwc Thanks for reaching out to us and reporting this issue. Azure.Storage.DataMovement is a library for uploading, downloading, and copying large data sets to and from Azure Blob storage.
Code that allows us to copy a file from a URL to an Azure Blob: https://learn.microsoft.com/en-us/azure/storage/common/storage-use-data-movement-library#copy-a-file-from-url-to-a-blob

Member name
Value
Description

SyncCopy
0
Download data from source to memory, and upload the data from memory to destination. Currently only available for copying from one Azure Storage resource to another.

ServiceSideAsyncCopy
1
Send a start copy request to Azure Storage to let it do the copying; monitor the copy operation progress until the copy is completed.

ServiceSideSyncCopy
2
Copy content of each chunk with with Put Block From URL, Append Block From URL, or Put Page From URL.

Refer the sample code here for parallel upload: https://learn.microsoft.com/en-us/azure/storage/common/storage-use-data-movement-library#set-the-number-of-parallel-operations
One important use case for this feature is when you need to move data from another cloud service (e.g. AWS) to Azure. As long as you have a URL that gives you access to the resource, you can easily move that resource into Azure Blobs by using the TransferManager.CopyAsync method. This method also introduces a CopyMethod parameter. The following table shows the available options for this parameter:
Member name	Value	Description
SyncCopy	0	Download data from source to memory, and upload the data from memory to destination. Currently only available for copying from one Azure Storage resource to another.
ServiceSideAsyncCopy	1	Send a start copy request to Azure Storage to let it do the copying; monitor the copy operation progress until the copy is completed.
ServiceSideSyncCopy	2	Copy content of each chunk with with Put Block From URL, Append Block From URL, or Put Page From URL.
Data Movement library has the ability to copy from one Azure Storage resource to another. Refer this: https://learn.microsoft.com/en-us/azure/storage/common/storage-use-data-movement-library#copy-a-blob
The full code sample is available on GitHub.

@navba-MSFT I don't see anywhere in what you shared that allows to pass a list of for example 1,000 urls to the library to have it tell azure to start downloading the urls. I only see the example you gave for copying a single url at a time.
The link to the parallel sample code doesn't show an example of doing a parallel transfer. It shows setting a property to the max parallel count, and then continue with the previous code and uploads single file with choices for source.

@johnwc Thanks for getting back. As you rightly pointed out from the documentation, it seems that Azure.Storage.DataMovement library does not have a built-in feature to upload files in bulk from a list of URLs. However, you can use the TransferManager.CopyAsync method to copy files from a URL to an Azure Blob. To upload multiple files, you can create a loop that iterates through the list of URLs and calls the TransferManager.CopyAsync method for each URL.
To optimize the upload process, you can set the number of parallel operations using the TransferManagerOptions class. This allows you to upload multiple files simultaneously, which can significantly improve the upload speed.
Here's an example code snippet that demonstrates how to upload multiple files from URLs using Azure.Storage.DataMovement library:
 static async Task Main(string[] args)
        {
            string connectionString = "<your-storage-account-connection-string>";
            string containerName = "<your-container-name>";
            List<string> urls = new List<string>
            {
                "https://example.com/image1.jpg",
                "https://example.com/image2.jpg",
                "https://example.com/image3.jpg"
            };

            BlobContainerClient containerClient = new BlobContainerClient(connectionString, containerName);
            await containerClient.CreateIfNotExistsAsync();

            TransferManagerOptions options = new TransferManagerOptions
            {
                MaximumConcurrency = 10 // Set the number of parallel operations
            };

            foreach (string url in urls)
            {
                Uri uri = new Uri(url);
                string blobName = uri.Segments[uri.Segments.Length - 1];

                BlobClient blobClient = containerClient.GetBlobClient(blobName);
                TransferContext context = new TransferContext();
                await TransferManager.CopyAsync(uri, blobClient, CopyMethod.ServiceSideAsyncCopy, null, context, options);
            }
        }

In this example, we first create a BlobContainerClient object using the storage account connection string and container name. We then create a TransferManagerOptions object and set the MaximumConcurrency property to 10 to allow up to 10 parallel operations.
Next, we loop through the list of URLs and for each URL, we create a Uri object and extract the file name from the URL. We then create a BlobClient object for the destination blob and call the TransferManager.CopyAsync method to copy the file from the URL to the blob. We pass in the TransferContext and TransferManagerOptions objects to control the transfer process.
Note that this is just an example and I haven't tested this yet. So you need to check if this serves your requirement and  you may need to modify the code to fit your specific requirements.

@navba-MSFT thank you for that, I will attempt to update our existing job to use DataMovement library. But, we do have to upload large sets of images, sometimes dealing with millions of urls. Is there a way to request a new endpoint be created that supports accepting a batch of urls instead of just one. That way we can submit batches of 100 instead of 1 at a time?

@johnwc Thanks for getting back.

we do have to upload large sets of images, sometimes dealing with millions of urls. Is there a way to request a new endpoint be created that supports accepting a batch of urls instead of just one. That way we can submit batches of 100 instead of 1 at a time?

@amnguye @seanmcc-msft Could you please help in tagging the right point of contact for DataMovement library to check if the above ask is feasible ?

It's definitely an interesting feature request, that's currently not feasible.
Also just to clarify the DataMovement library sample that @navba-MSFT provided is from the Microsoft.Azure.Storage.DataMovement package and not the Azure.Storage.DataMovement package
But let me try to vet this scenario correctly, so the Azure.Storage.DataMovement package can support it in the future. So you have a bunch of URLs that live outside of Azure Storage, and you want to do a service copy for each URL to one container in a storage account correct?
It's probably not feasible right now in the current design of the DataMovement Library. We would probably have to figure out the best API to make that would take in non-azure storage URLs, then on top of that many different URLs as well.
Probably multiple work items here that need to be done:

Support for non-Azure URLs (this would require to make a new StorageResource type object to wrap around public URLs that do not live in an Azure Storage Account
Taking in multiple StorageResource objects that are not apart of the same StorageResourceContainer (e.g. the multiple URLs that do not live in the same container)
Ensuring this works along side the pause and resume feature

Also just to clarify the DataMovement library sample that @navba-MSFT provided is from the Microsoft.Azure.Storage.DataMovement package and not the Azure.Storage.DataMovement package

Can the sample code be used the same with Azure.Storage.DataMovement if just to be updated with correct config+method calls, etc?

So you have a bunch of URLs that live outside of Azure Storage, and you want to do a service copy for each URL to one container in a storage account correct?

Yes, do a copy blob, so that the calling code doesn't have to wait on it. (We have storage event hooks setup to know when the file reached the container.) This already is supported today, just not for a batch of urls.

It's probably not feasible right now in the current design of the DataMovement Library. We would probably have to figure out the best API to make that would take in non-azure storage URLs, then on top of that many different URLs as well.

That's what this request was requesting, for there to be a azure api endpoint added that supported accepting more than one Url for StartCopy. That then DataMovement can utilize. Again, the existing API already supports copying a single remote url asynchronously.

Support for non-Azure URLs (this would require to make a new StorageResource type object to wrap around public URLs that do not live in an Azure Storage Account

I don't understand this statement, as the existing Azure Storage API supports copying from a arbitrary url asynchronously along with the SDK.

Ensuring this works along side the pause and resume feature

I don't think this functionality would be doable, as the existing API only supports checking status and aborting a copy. We wouldn't need this personally, as for us, once we push the copy request to azure we would never need to pause the copying in azure.

Can the sample code be used the same with Azure.Storage.DataMovement if just to be updated with correct config+method calls, etc?

I currently can't provide a code snippet on how to do this using the Azure.Storage.DataMovement package because we don't yet support public URLs being copied into Azure Storage. This though could be easily done using the base Azure.Storage.Blobs package.
string connectionString = "<your-storage-account-connection-string>";
            string containerName = "<your-container-name>";
            List<string> urls = new List<string>
            {
                "https://example.com/image1.jpg",
                "https://example.com/image2.jpg",
                "https://example.com/image3.jpg"
            };

            BlobContainerClient containerClient = new BlobContainerClient(connectionString, containerName);
            await containerClient.CreateIfNotExistsAsync();

            TransferManagerOptions options = new TransferManagerOptions
            {
                MaximumConcurrency = 10 // Set the number of parallel operations
            };

            foreach (string url in urls)
            {
                Uri uri = new Uri(url);
                string blobName = uri.Segments[uri.Segments.Length - 1];

                BlobClient destinationClient = containerClient.GetBlobClient(blobName);
                TransferContext context = new TransferContext();
                CopyFromUriOperation copyOperation = await destinationClient.StartCopyFromUriAsync(uri);
                // if you want to wait on the copy operation completing
                copyOperation.WaitForCompletion();
            }

Support for non-Azure URLs (this would require to make a new StorageResource type object to wrap around public URLs that do not live in an Azure Storage Account

I don't understand this statement, as the existing Azure Storage API supports copying from a arbitrary url asynchronously along with the SDK.

Just to clarify, me making that work item list is more for me to understand and scope how much work there is to do here.
We are looking to support every scenario possible. Currently we do not support this in the DataMovement package but it is supported in the Azure.Storage.Blobs package. (It's what I used in the sample above)

I don't think this functionality would be doable, as the existing API only supports checking status and aborting a copy. We wouldn't need this personally, as for us, once we push the copy request to azure we would never need to pause the copying in azure.

Pause and Resume is a client side feature, not a REST API feature. On our side, we need to make sure once we support this feature we rule out what would happen in this scenario. We would most likely not abort the copy since it's a service side async copy. However if it's multiple URLs being copied, we need to at least save the state of which URLs have been copied so far and which ones haven't been copied yet (e.g. if you paused and not every URL copy has been started yet and then resumed and copied the rest of the URLs). This particular feature may not be apart of your scenario however I can see it being apart of other scenarios, and I'm just trying to cover all our bases here.

What we use currently today is the Azure.Storage.Blobs package, it only supporting a single URL at a time is what prompted us to open this request. Having to transfer so many file urls one at a time will take us a month or more.


Multi Mikrotik

Hi all, its posible add multiple mikrotiks?

No I don't think it's possible with the current form. An easy was would be to duplicate the docker container with a separate configuration. But then again I never tested it so there might be some collision of some sort in influxdb. I'll take a look this weekend on what is best to do in this case.


Token.Expiry not Sql 2005 compatible

Thought I would post a question to determine whether a decision was made to purposely support MSSQL 2008 and later.  I am having to support a minimum of MSSQL 2005 where datetimeoffset is not a supported type.  See https://msdn.microsoft.com/en-gb/library/bb630289.aspx
The entity Token has a property called Expiry of the .NET type DateTimeOffset.
See https://github.com/IdentityServer/IdentityServer3.EntityFramework/blob/master/Source/Core.EntityFramework/Entities/Token.cs#L42
Have you guys got experience with EF with how I can use this library against MSSQL 2005.

I just saw an email saying Sql2005 end support is coming soon. I know that's unrelated to your question, but I thought I'd mention it.

Yeah, you're right 12 April 2016.

So I'd find out first if your company still needs/wants to work with Sql2005 before we try to do something in IdSvr. My gut feeling is that since we're already committed to DateTimeOffset that I don't foresee changing that.

Any update?

Yeah,  long story short is that I want this to work with Sql2005.  My initial thoughts are to fork this library locally and to map Token.Expiry to 2 columns (DateTimeUtc, Offset).  Any thoughts as to why Token.Expiry is a DateTimeOffset and not just a UTC DateTime?

DateTimeOffset is the replacement for it -- it has a superset of features.
So yea, you'll have to map to Sql2005 and then map back to DateTimeOffSet for IdSvr. Sorry.

Unofrunately, I have to support SQL Server 2005 as well. So I had to add a fix https://github.com/mnaoumov/IdentityServer3.EntityFramework/commit/6405f0e0cec94c36b61e1aa77580020bc33262ae


Reformatting script for codet5-base.gson.mid-599.idx-29543.2.mutant

See human reformatting here

LGTM!


Makefile

Please see commit message.

Files have changed since this patch was written. Probably some whitespace tidying should be checked in at some point.


[SPARK-39406][PYTHON] Accept NumPy array in createDataFrame

What changes were proposed in this pull request?
Accept NunPy array in createDataFrame, with existing dtypes support.
Note that

by the constraint of Spark DataFrame, we support 1-dimensional and 2-dimensional arrays only.
full NunPy <> PySpark mappings will be introduced as a follow-up.

Why are the changes needed?
As part of SPARK-39405 for NumPy support in SQL.
Does this PR introduce any user-facing change?
Yes, NumPy array is accepted in createDataFrame now:
Before:
>>> spark.createDataFrame(np.array([[1, 2], [3, 4]]))
Traceback (most recent call last):
...
TypeError: Can not infer schema for type: <class 'numpy.ndarray'>

>>> spark.createDataFrame(np.array([0.1, 0.2]))
Traceback (most recent call last):
...
TypeError: Can not infer schema for type: <class 'numpy.float64'>

After:
>>> spark.createDataFrame(np.array([[1, 2], [3, 4]])).show()
+---+---+
| _1| _2|
+---+---+
|  1|  2|
|  3|  4|
+---+---+

>>> spark.createDataFrame(np.array([0.1, 0.2])).show()
+-----+
|value|
+-----+
|  0.1|
|  0.2|
+-----+

How was this patch tested?
Unit tests.

Seems like it would fixed when https://github.com/apache/spark/pull/36813 is merged.
Let's rebase after then.

Otherwise looks good if test passes

Rebased to pass tests. Thanks @itholic!

Rebased for conflicts in python/docs/source/getting_started/install.rst only.

Merged to master.


High CPU load (even when idle)

HippoPlayer creates a high CPU load for me (constantly maxing out on of my CPUs), even if no track is playing.

I see. Is this after playing a track or just once the program has been started?

This is also the case for me right after starting the program with fresh config files and empty playlist.

Ok thanks! I will investigate.

This seems to have resolved itself in current versions, by the way.

ah, odd. I looked at some profiling on windows but I couldn't find any high-cpu usage. Will close this one for now and we can re-open if it happens again.


Spurious protein terms

I was not able to find a couple of terms that I think would be useful additions to SO. The protein sequence databases contain an unknown but significant number of spurious proteins. These are not translated in nature.  In general spurious proteins are created by automated gene calling pipeline errors. My group has been involved in producing software and databases to identify and track these (PMID:29721311 and PMID:22434837). I would suggest a general term of "Spurious protein" that could perhaps live in polymer_attribute.  A specific type of spurious protein is known as a Shadow ORF. These are generally found on the opposite strand of a known gene. I would suggest that "Shadow ORF" as a new term that lives under the "Spurious protein" term. It is worth noting that shadow ORFs have been found on the opposite strand to non-coding RNA genes such as rRNA.

Hi @Alexbateman1,
I am sorry it has taken so long to get to this request. I agree that this term would be relevant. However, I do not know that it belongs under polymer_attribute. I think it might be better suited under this term:
SO:0001410,  experimental_feature, def: "A region which is the result of some arbitrary experimental procedure. The procedure may be carried out with biological material or inside a computer."
What do you think?
As for the articles that you have added, I think they are important papers, but I do not think they should be included in the SO as they are for specific programs rather than papers about the terms themselves. I found one paper that is pretty good for describing suprious proteins, PMID:21771858. However, I cannot find any papers that mention shadow ORF except for the papers that you have provided. Are you aware of any publications that specifically mention shadow ORF that are not papers specifically about programs for predicting them?
One more thing I wanted to ask about was a specific definition. Here is what I have, but this may need changing before I add it to the so.
spurious_protein: def: "A region of DNA that is predicted to be translated and transcribed into a protein by a protein detection algorithm that does not get transcribed in nature."
Can you please give me a specific definition for shadow_orf?
Thanks,
Dave

Hi @Alexbateman1,
The new term spurious_protein (SO:0002248) has been added to the SO. The SO Browser will be updated with the new term within about 24 hours.
Thanks,
Dave


Load subclasses of FileDepotFtpAnonymous

Followup to #1753

@Fryguy @himdel Please review

@brandondunne Does anything actually inherit from FileDepotFtpAnonymous? If not, I don't see a point...
IMHO it should either be both in FileDepotFtpAnonymous and all the other leaf FileDepot subclasses - just in case - or in neither.

@himdel I agree that it felt funny to put this everywhere.
I was reviewing this change and thought we could recursively call this method for the subclasses that we just loaded.  I updated the PR to reflect that.  It works fine with the failure @jrafanie pointed out here and seems to work fine in console as well.
cc @Fryguy

@brandondunne OK, works for me, both with the original reload bug and with @jrafanie's test, and I actually like it better than putting it in FileFtp..
Only thing.. the comment I put above eager_load... mentions direct subclasses so please update it to reflect the change?

@himdel updated the comment based on your suggestion

Oh nice I like this.

I love how initial PRs morph and change with feedback into something we're all much happier with.  :heart:  Thanks @himdel @brandondunne

@brandondunne What does the initial require of FileDepot which then recursively requires the subclasses in the right order?

Something will end up calling one of the classes or subclasses, which will encounter the line FileDepotWhatever < FileDepot which will load FileDepot.  require_dependency is properly reentrant.

Yeah, looks good.  :star2:


general question about your experience building this project

Hi @Imira-S-R  ,
Hope you are all well !
I would like to create a clone of teampass a collaborative password manager, and I was wondering if you knew or can explain how to create a catalog of items, organised with an animated tree view, and flutter_rbac for managing roles and permissions. From your experience, what advices would you give me for neat architecture.
Thanks for any insights or inputs on that question.
teampass screeenshot:

Cheers,
Luc

Hello @lucmichalski 👋
Sorry for the delayed response
I'm still a student learning Flutter, and I haven't implemented Role-based access control because I only wanted to make a simple password manager. You may, however, use this package for Animated Tree View.
To have a clean architecture, the app must be efficient, the code must be understandable, and the app must be well tested in order to reduce the amount of problems in the final product. These are some of the factors I consider while creating an app.
I hope this knowledge is useful to you.
Imira


Unified logging interface

At the moment the library uses the Lambda Context, where it is available, to access the Logger object. This doesn't scale well and results in "hidden" exceptions in a few places. We should introduce a logger

Moved to slf4j-api in the latest commit.

About to merge changes into master, resolving.


[Request] Separate grunt and gulp plugins from core

I don't use grunt and gulp, but this includes them.
They significantly increase file size.
Can you separate them to other package?

We bundled them together very intentionally. With npm, it's very hard to manage the compiler version separate from the plugins. Most users care most about the compiler version and just want the working plugin version.
Also, future work will further integrate the functionality. Grunt already uses the gulp plugin internally.

@ChadKillingsworth

With npm, it's very hard to manage the compiler version separate from the plugins. Most users care most about the compiler version and just want the working plugin version.

Yes, and monorepo with lerna is a solution for the situation.
This is my fork using lerna.
https://github.com/teppeis/closure-compiler-npm/tree/monorepo
You can publish all packages with lerna publish at a time and there are no breaking changes for existing users.
The current google-closure-compiler package has 237 dependencies in production, but new google-closure-compiler-jar has nothing. The 237 packages are unnecessary for non-gulp/grunt users like webpack plugins or full-scratch tool.
If this is interesting, I'll send a pull-request.

We really don't want to maintain extra packages right now.
However the dependencies were somewhat out-of-control. I've PR'd a change to improve the situation.

@ChadKillingsworth Thanks! It's reasonable improvement.


Endpoint to get User Information

Is your feature request related to a problem? Please describe.
When developing an application that will authenticate with oauth to visualizer.coffee, it will be useful to have an endpoint that shares some information about the user, such as a unique ID, their name, their profile image, and (maybe, if the user grants access) their email. Email is only really important if a third party is developing an app where maybe in the future they want to move off of visualizer.coffee oauth (maybe the service goes away?)
Describe the solution you'd like
/oauth/user (or whatever) endpoint that allows you to authenticate, and returns JSON, something like...
{"name": "name", "email"<EMAIL_ADDRESS>etc..
Describe alternatives you've considered
Not sure if there are any!

You now have /api/me endpoint for this. Lemme know if it doesn't work as expected.


Remove macos 13

It keeps failing as brew cannot install graphviz without also installing python 3.12 which conflict with the existing python install when linking 2to3.
I don't think it is worth testing this on that many macos and linux platform, as it's mostly a theme, so the risk of having a production system fail because of a bug on macos is low.

Thanks, that was fast !


WildFly 10 - How to change default encoding?

Hi,
I have a question. How can I configure WildFly / Undertow to set UTF-8 as default response encoding?
I don't want to add ";charset=UTF-8" to every JAX-RS API therefore I am searching for a generic solution.
I ran these commands, but they don't have the right effect.
$WILDFLY_HOME/bin/./jboss-cli.sh --connect --command="/subsystem=undertow/servlet-container=default:write-attribute(name=use-listener-encoding,value=true)" > config.log 2>&1 &
$WILDFLY_HOME/bin/./jboss-cli.sh --connect --command="/subsystem=undertow/servlet-container=default:write-attribute(name=default-encoding,value=UTF-8)" > config.log 2>&1 &

Does anyone know, how can I fix this?
Thank you in advance.
Niklas

Please use forums at https://developer.jboss.org/en/wildfly

@ctomc Thank you. I created a discussion.


XNYS - incorrect start/end times using '.is_open_on_minute' function?

Hi,
I think I may have found an error with the start and end times for XNYS when using .is_open_on_minute to return a boolean.
Reading through the core trading session hours for the NYSE, it seems to me that on a regular trading day the market starts at exactly 9:30:00 EST (inclusive) and is closed at exactly 16:00:00 EST (inclusive). I wrote some code to test some times and I am getting conflicting answers.
Currently,

9:30:00 returns market closed. I think this should be open.
9:30:01 returns market closed. I think this should be open.
9:31:00 is the first time returned as open.
16:00:00 returns market open. I think this should be closed.

My code attached.
Thanks in advance.
import pandas as pd
import pytz
import trading_calendars

NY = 'America/New_York'
us_calendar = trading_calendars.get_calendar('XNYS')

#Market on Friday at 9:30:00 EST
pd_dt1 = pd.Timestamp(year=2020, month=10, day=9, hour=9, minute=30, second=0, tz=pytz.timezone(NY))
open_boolean1 = us_calendar.is_open_on_minute(pd_dt1)
print("Friday at {} EST. open_boolean should be True: {}".format(pd_dt1.strftime('%H:%M:%S'), open_boolean1))

#Market on Friday at 9:30:01 EST
pd_dt2 = pd.Timestamp(year=2020, month=10, day=9, hour=9, minute=30, second=1, tz=pytz.timezone(NY))
open_boolean2 = us_calendar.is_open_on_minute(pd_dt2)
print("Friday at {} EST. open_boolean should be True: {}".format(pd_dt2.strftime('%H:%M:%S'), open_boolean2))

#Market on Friday at 9:31:0 EST
pd_dt3 = pd.Timestamp(year=2020, month=10, day=9, hour=9, minute=31, second=0, tz=pytz.timezone(NY))
open_boolean3 = us_calendar.is_open_on_minute(pd_dt3)
print("Friday at {} EST. open_boolean should be True: {}".format(pd_dt3.strftime('%H:%M:%S'), open_boolean3))

#Market on Friday at 16:00:00 EST
pd_dt4 = pd.Timestamp(year=2020, month=10, day=9, hour=16, minute=0, second=0, tz=pytz.timezone(NY))
open_boolean4 = us_calendar.is_open_on_minute(pd_dt4)
print("Friday at {} EST. open_boolean should be False: {}".format(pd_dt4.strftime('%H:%M:%S'), open_boolean4))

#Market on Friday at 16:00:01 EST
pd_dt5 = pd.Timestamp(year=2020, month=10, day=9, hour=16, minute=0, second=1, tz=pytz.timezone(NY))
open_boolean5 = us_calendar.is_open_on_minute(pd_dt5)
print("Friday at {} EST. open_boolean should be False: {}".format(pd_dt5.strftime('%H:%M:%S'), open_boolean5))

#Market on Saturday at 9:30:00 EST - Should be closed
pd_dt6 = pd.Timestamp(year=2020, month=10, day=10, hour=9, minute=30, second=0, tz=pytz.timezone(NY))
open_boolean6 = us_calendar.is_open_on_minute(pd_dt6)
print("Saturday at {} EST. open_boolean should be False: {}".format(pd_dt6.strftime('%H:%M:%S'), open_boolean6))

I would agree that 9:30:00 and all times between 9:30:00 and 9:31:00 are open for trading. I would agree that trading_calendars should include those times as open.
As for 16:00:00 I would argue that is also a valid trading time because the NYSE offers the MOC/LOC facility and those trades occur on the 16:00:00 bell.

@rsheftel the MOC/LOC for 16:00:00 logic makes sense to me. I think your assumption is fair and agree on your fix suggestion.
To add to the above. I tried the .open_and_close_for_session command and it seems to have similar/same results as those in my initial query.
import datetime
import trading_calendars

#Get datetime now
today_datetime = datetime.datetime.now()

#Get XNYS calendar
us_calendar = trading_calendars.get_calendar('XNYS')

#Retrieve open and close times for today
us_calendar.open_and_close_for_session(today_datetime.strftime("%Y-%m-%d"))

#Results
#(Timestamp('2020-10-09 13:31:00+0000', tz='UTC'),
# Timestamp('2020-10-09 20:00:00+0000', tz='UTC'))

If you look in the definition of the XNYS class you can see line #164 defines
open_times = ((None, time(9, 31)),)
https://github.com/quantopian/trading_calendars/blob/master/trading_calendars/exchange_calendar_xnys.py

Yep - as @rsheftel the open+1 minute behavior exists as an artifact of when trading_calendars was more tightly interwoven with zipline (https://github.com/quantopian/zipline) and we wanted the market to only indicate as open on the completion of the first minute bar.
I agree that for the majority of use cases this isn't quite the desired behavior. @richafrank and I have talked about changing it in the past, but haven't gotten around to it yet.
I should address this when I go fix up the documentation for trading_calendars though.
I'm closing as this is the currently expected behavior, though I agree it is initially confusing.

@gerrymanoim thanks for confirming that these functions are working as currently intended to. I too agree that it probably isn't the desired behavior in most cases.
Is the plan to change the behavior of this function in the future to reflect the exact opening time or to update the documentation to explain the expected current behavior?

My first plan is to update the docs to clarify the behavior (we have a documentation sprint coming soon).
In the future I'd like to get the actual behavior of this function changed, but that's further away.


Datetime not displayed properly for min/max attributes

Short description of the problem:
Datetime is not rendered properly when using min/max attributes.
What behavior are you expecting?
The Day, Month and year should be displayed properly when using min/max attributes.
For example, when providing min date as 2016-04-12, the date and month is not displayed on clicking the date selection. Once you scroll the date/month, it renders correctly.
Once the user selects Done without changing anything, the date 2016-01-01 is saved in the model even though this is outside the range provided in min and max.
This is happening in browser and iOS
Which Ionic Version? 1.x or 2.x
2.0.0-beta.10
Plunker that shows an example of your issue
https://plnkr.co/edit/PKNPDQoQiC0x20eGl2Z0?p=preview
Ionic CLI Version: 2.0.0-beta.32
Ionic App Lib Version: 2.0.0-beta.18
OS: Windows 7
Node Version: v5.8.0

Hello, thanks for opening an issue with us! I am going to be closing this issue as a duplicate of https://github.com/driftyco/ionic/issues/6850. Thanks for using Ionic!


[release-1.22] Bump containerd to v1.5.16-k3s1-1-22

Proposed Changes
Bump containerd to v1.5.16-k3s1-v1-22
Types of Changes
version bump
Verification
Check containerd version
Testing
Linked Issues

User-Facing Change
The embedded containerd version has been bumped to v1.5.16

Further Comments

Sonobuoy tests flaked under load; merging.


ForeignEntity property name is being generated with the entity name vs the name

I have an entity defined with foreign entities, when the code is generated it is ignoring the property name and using the foreignentityname.
Steps to reproduce
-yaml

Name: Advertiser
TableName: Advertiser
Features:
- Type: GetRecord
- Type: GetList
- Type: DeleteRecord
IsProtected: true
PermissionName: candelete
- Type: UpdateRecord
IsProtected: true
PermissionName: canupdate
- Type: PatchRecord
IsProtected: true
PermissionName: canupdate
- Type: AddRecord
IsProtected: true
PermissionName: canadd
Properties:
- Name: Name
Type: string
CanFilter: true
CanSort: true
- Name: CreatedBy
Type: User
ForeignEntityName: User
CanFilter: true
CanSort: true
- Name: ModifiedBy
Type: User
ForeignEntityName: User
CanFilter: true
CanSort: true

##Output
using Ordering.Domain.Users;
using Ordering.Domain.Users;
[Table("Advertiser")]
public class Advertiser : BaseEntity
{
[Sieve(CanFilter = true, CanSort = true)]
public virtual string Name { get; private set; }
[JsonIgnore]
[IgnoreDataMember]
[Sieve(CanFilter = true, CanSort = true)]
public virtual User User { get; private set; }

[JsonIgnore]
[IgnoreDataMember]
[Sieve(CanFilter = true, CanSort = true)]
public virtual User User { get; private set; }

hmm, interesting. Will try and take a look at this later tonight!

About to head out to an event, so can't dig in deep at the moment, but at first glance you might want to try something like this:
      - Name: CreatedById
        Type: Guid
        ForeignEntityName: User
        ForeignEntityPlural: Users
      - Name: ModifiedById
        Type: Guid
        ForeignEntityName: User
        ForeignEntityPlural: Users

If I'm remembering that part of the codebase correctly, it needs the Id property and will make the entity based on off of the given info automatically something like this with just the entity might not work at the moment. Can think though options to make things more flexible if that is the case though.

able to repro and yeah it shouldn't be happening that way. digging now!

found it. thanks for the callout!


Comment up Chef::Client and privatize/deprecate unused things

@lamont-granquist based on our talk last week, the main question was whether we should deprecate the attr_writers in question.  I have deprecated ohai= and rest=, and deprecated runner entirely; we'll find out soon enough if there are any real use cases, but it seems unlikely.  I left node alone simply because tests use it.
If that's all OK, I'mma merge.

tests are pissed, probably needs a rebase?

@chef/client-core this needs review

@lamont-granquist the test failures were a bad rebase, actually,  I've fixed it up.  Thanks :)

:+1:


Get degree from encoder produces KeyError if no motor was used before on ev3dev

If one creates a program that simply reads out the encoder value and shows it, then this program will crash because an actor is not added in the configuration of ev3dev, since a motor actor was not used in the program.

So an actor should be added even if no actor is used for this particular case.

Shouldn't the UsedHardware visitor check this?

Looks like a job for it. I am working on a similar topic, so I will come across this at some point, I think.


How to pass multiple Subscription IDs

Whilst the ManagementGroup option isn't working. I thought it might be an idea to pass multiple subscription IDs as your instructions imply that this is possible. Could you advise how this is done? Do they need to be comma separated? Space separated? Or something else?

Hello @mark4409
I believe the correct form is:
-SubscriptionID "00000000-0000-0000-0000-000000000000","11111111-1111-1111-1111-111111111111"


CPU dispatch (kinda)

I can't get generic to work, because it still sometimes seems to call AVX2 Eigen stuff even in
the generic path, and the AVX512 path always runs because of some static init, but it's a start.

Removing static initialization and forcing Eigen to be 64 byte aligned seems to be sufficient. Merging for now, though might still be iffy.


Embedding and multiple threads

Hi,
I am interested in how to integrate RustPython so that scripts can be called by multiple threads. Am I right in thinking that each thread would need to run it's own interpreter or can one be shared with each thread having a different scope?

You can look at vm/src/stdlib/thread.rs to see how threads are implemented internally; are you looking to enter into the vm from a thread that you started? e.g.
let vm = ...;
for i in 0..10 {
    let vm = vm.clone(); // or something
    let userfunc = userfunc.clone();
    thread::spawn(move || vm.invoke(&userfunc, vec![vm.ctx.new_pyobj(i)]));
}

Something along those lines? Just trying to figure out how specifically you want to do this, I think we probably could provide a vm.start_thread(impl FnOnce(&VirtualMachine) -> R) -> JoinHandle<R>

I want to use Python scripts as an ETL interface so that users could write scripts and then execute as required. Steps would be:

User writes script and compiles it.
User or scheduled job executes the script at a later time.

I am not sure an interface is required but I an interested in what can be shared (or NOT). Ideally the compiled scripts and global scope with built-in and custom functions could be created once and shared with locals being set on each execution.

Yep, it's totally possible to share builtins/modules/globals between threads. What I meant by interface is just a vm.start_thread() method with that signature. If that was available to you (with the vm passed to the closure having the same builtins and stuff as the vm on the main thread) do you think that would make your use case possible? You could share globals by just taking a function from the defined scope, e.g.
foo = 1
def bar():
    global foo
    foo += 1

If you get the bar object from that scope and vm.invoke() it will properly have access to the globals it expects to.

Yep, start_thread would work, that make it a very simple process and seams safer than doing it yourself outside of RustPython.

Hi @coolreader18,
Thanks for being so responsive!
What are the steps to do it with current code base? I am trying to understand how the vm, scope and compiled statements tie together. There is lots of things going on in the code base so it is hard for me to understand. For example the PyNativeFunc is created via vm.ctx.new_function, does that mean that function can only be used with that vm?
In terms of scope, how do load all of my standard stuff into globals and reuse that with locals? The API implies that the scope is tied to the vm. Is below the correct way to reuse the globals (or is the parent scope below mutable):
// Create the parent scope with globals
let scope = vm.new_scope_with_builtins();
// Set the add function in global scope
scope.globals.set_item("add", vm.ctx.new_function(add), vm).expect("failed");
// Create a child scope with locals
let scope_child = scope.new_child_scope(&vm.ctx);
scope_child.get_locals().set_item("a", vm.new_pyobj(40.0), vm).expect("failed");
scope_child.get_locals().set_item("b", vm.new_pyobj(2.0), vm).expect("failed");
// Run a script with scope_child...

Also in terms of overhead what is the cost in creating a vm and scopes?

Ah, yeah, I can see how that might be confusing. You can think of the vm passed to the function for start_thread as the same vm, or rather, the same interpreter; objects can move between them in any way you want and the builtins, sysmodule, ctx, etc fields are all the same as the original vm.

The scope isn't tied to the vm, we just use the vm as a catch-all for any functions that require access to the vm. The parent scope is mutable, but yeah, that's the correct way to do it. And in terms of overhead, it should be pretty quick; you can see the overhead of the thread vm creation; it's mostly just cloning a few Arcs. And dicts are created all the time in python, and that's all a scope is, pretty much.


No support for old bitcoind versions?

Looks like some APIs (ImportAddressRescan in particular) are not backwards compatible with old versions of bitcoind. I can't find a good way to add compatibility without breaking changes. Any ideas?

You could always add a method with a _v016 suffix or so..
On Wed, Oct 17, 2018, 02:09 gracenoah<EMAIL_ADDRESS>wrote:

Looks like some APIs (ImportAddressRescan in particular) are not backwards
compatible with old versions of bitcoind. I can't find a good way to add
compatibility without breaking changes. Any ideas?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stevenroose/go-bitcoin-core-rpc/issues/2, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0F3O1zE_Zd9-MNzYVPjJV2FFbVkWSIks5uloNQgaJpZM4Xi8bW
.

I tired, but it would require a bigger refactor because you can't have two methods registered for the same RPC method :(

Is that really true (been a long time since code base)?? Because you have the "verbose" versions etc, no? Like different methods for different arguments provided...
Perhaps you mean one Cmd type per official method?
Perhaps having a branch per main version (like v0.15) would be an idea? But then the user needs to know upfront what version of Bitcoin Core he'd be dealing with.

Yeah, the issue is with the Cmd stuff.
Yep, I'm using a branch right now and I've updated the one API that I'm using that doesn't work. Another option is to always use the "old" version with the empty account. IIRC the new version of bitcoind is backwards compatible with the old API as long as the account string is empty.

Sorry, on phone so can't look into the source. But why exactly is it one
Cmd per command string? Is that map actually still used?
On Wed, Oct 17, 2018, 22:33 gracenoah<EMAIL_ADDRESS>wrote:

Yeah, the issue is with the Cmd stuff.
Yep, I'm using a branch right now and I've updated the one API that I'm
using that doesn't work. Another option is to always use the "old" version
with the empty account. IIRC the new version of bitcoind is backwards
compatible with the old API as long as the account string is empty.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stevenroose/go-bitcoin-core-rpc/issues/2#issuecomment-430797379,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0F3NhKh14XIPCKumSLHwx7bN8Jfvr-ks5ul6IlgaJpZM4Xi8bW
.


const and let

Prefer const and let over var, everywhere.
https://github.com/airbnb/javascript#references--prefer-const

Closed via #36


Fetching data page missing in the docs

Describe the bug
https://docs.solidjs.com/guides/fetching-data leades to 404.
This is a crucial info and it's shame that is missing from the docs.
Please fix it.
Your Example Website or App
https://docs.solidjs.com/guides/fetching-data
Steps to Reproduce the Bug or Issue
Go to https://docs.solidjs.com/guides/fetching-data
Expected behavior
Showing fetching data info.
Screenshots or Videos
No response
Platform

Browser: All

Additional context
No response

shrubway It's not showing 404 for me, could you check again?

Yeah I wasn't able to reproduce yesterday or today. It may have been some some odd caching/deployment timing thing.


Add: base

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #50   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          17     17
  Lines         205    205
=====================================
  Hits          205    205

Impacted Files
Coverage Δ

...n/merge-two-sorted-lists/merge_two_sorted_lists.go
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Please add your extension in open source VSCodium marketplace

As VSCodium is VS Code without the telemetry, it'd be great to see BuilderIO into the extensions marketplace:
https://github.com/VSCodium/vscodium/blob/master/DOCS.md#extensions-marketplace

The plugin work well, it is just a matter to push it into the marketplace

How did you get it into VSCodium?
I don't see it in mine. I'm on Ubuntu, but it's not related.

Thanks @Imred-Gemu
Noscript was blocking VSCode marketplace so hard, I didn't see the download link :)
I have an error when trying to launch the "open Builder.io on the side":
Command 'Builder.io: Open Builder.io visual editor to the side' resulted in an error (Running the contributed command: 'builder.openJsxLiteEditorToTheSide' failed.)
And open Builder.io: Oh no! We had an error :( Try refreshing this page and chat us or contact<EMAIL_ADDRESS>if this continues
Well, if it doesn"t launch, I guess we'll have to wait a little bit for a collaborator.
Updating the title.


Fix links in RayTrain readme

Followup to https://github.com/GoogleCloudPlatform/ai-on-gke/pull/87

/assign @yiyinglovecoding

/cc @yiyinglovecoding


koord-descheduler: migrationController set default evictQPS/evictBurst

Signed-off-by: Joseph<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/hold

/hold cancel

/lgtm

/lgtm


Document deterministic and verifiable builds

#100 introduces the ability to verify on-chain programs against locally compiled source. This should be added to the documentation.

https://github.com/project-serum/anchor/pull/501


416 (Requested Range Not Satisfiable)

Hi guys,
I am working on an Audio Player in Meteor which uses CFS for uploading audio and image of audio. I can get the image back from CFS very well but for the audio file, which is kinda big, i get back:
GET urlhere 416 (Requested Range Not Satisfiable)
My guess is that it doesn't work with big files. Any ideas?

cfs is designed to handle large files. Did you get anywhere with this?

Not really, I moved on doing the other parts of the application.
Once I feel safe, I will come back to this and let you guys know what I have discovered.

I had same problem, for now, current master normally upload files. If you use nginx, you need change storage chunksize to 1Mb (nginx default).

This was it. Thank you so much!


Update to Airlift 0.194

Can you add a note to the commit message indicating how it affects Presto? Also, maybe link to the issues it fixes, such as the t-digest index out of bounds bug.

Updated description and added link.

can this get in 332 release?

@tooptoop4 yes, it will be in 332


Drola generalized

Ther verion upto this pull requet contains all the main functionalities. Bi directional AES ecured communication between sensor nodes and Application server through LoRa Gateways.

Updated ReadMe file


NFA Regex Engine does not consider combining chars properly

Steps to reproduce

vim --clean
paste the following text:

the quick brown fox
quisdjdfsdfsdf quisdjdfsdfsdf
t̶h̶e̶ ̶q̶u̶i̶c̶k̶ ̶b̶r̶o̶w̶n̶ ̶f̶o̶x̶
q̶u̶i̶s̶d̶j̶d̶f̶s̶d̶f̶s̶d̶f̶ ̶q̶u̶i̶s̶d̶j̶d̶f̶s̶d̶f̶s̶d̶f̶

Try to search the strike-through characters using the default regex engine: /\%u336 (does not match any characters)
Force searching using the old regex-engine: /\%#=1\%u336  (matches as expected)

Note line 3 and 4 contain the combining char
U+0336 Dec:822 COMBINING LONG STROKE OVERLAY (Mn) ̶ /\%u336 "\u0336"  for each base character.
Noticed at: https://vi.stackexchange.com/questions/42474/do-not-red-underline-unicode-control-points-as-misspelled-words/42478#42478
Expected behaviour
Search should match, regardless of the selected regex engine.
Version of Vim
9.0.1392
Environment
Windows 11,
Logs and stack traces
No response

I confirm (I broke my head until I realized what was going on)
So, абракада́бра (аbracadabra)
where а́ — <а> 1072, Hex 0430, Oct 2060, Digr a= < ́> 769, Hex 0301, Octal 1401
let's try to find it:
/\%#=1\%d769 — match
/\%#=2\%d769 — not match
/\%#=1а\%d769 — not match
/\%#=2а\%d769 — match
/\%#=1\%u0301 — match
/\%#=2\%u0301 — not match
/\%#=1а\%u0301 — not match
/\%#=2а\%u0301 — match
/\%#=1\%Cа́ — match
/\%#=2\%Cа́ — match
version gvim 9.0.1603


fix check point image recognition

Summary
Issue
Running boxen --disk <check_point_image> command fails with:
failed gleaning source data from disk

This PR adds a fix by:

Adding required  Vendor Platform Map values
Adding required Disk Version Map values

LGTM @pouriyajamshidi thanks for the fixup!


feat(rpc): improve failure messages for Filecoin.EthEstimateGas

Issue summary

Lotus uses some Filecoin.EthCall functionalities that have yet to be implemented in forest to make the failure messages more user-friendly for Filecoin.EthEstimateGas.  We should close the gap once Filecoin.EthCall is implemented
e.g.
2024-07-17T14:53:44.6298673Z | Filecoin.EthEstimateGas                           | Rejected("Message has no gas limit set")                                                            | Rejected("failed to estimate gas: message execution failed: exit 33, revert reason: 0x5416eb985904247900000000000000000000000000000000000000000000000000000000, vm error: message failed with backtrace:\n00: f0131541 (method<PHONE_NUMBER>) -- contract reverted (33)\n (RetCode=33)") |
2024-07-17T14:53:44.6304762Z | Filecoin.EthEstimateGas                           | Rejected("Message has no gas limit set")                                                            | Rejected("failed to estimate gas: message execution failed: exit 33, revert reason: none, vm error: message failed with backtrace:\n00: f0120962 (method<PHONE_NUMBER>) -- contract reverted (33)\n (RetCode=33)")                                                                       |
2024-07-17T14:53:44.6310764Z | Filecoin.EthEstimateGas (3)                       | Rejected("Message has no gas limit set")                                                            | Rejected("failed to estimate gas: message execution failed: exit ErrUnhandledMessage(22), revert reason: none, vm error: message failed with backtrace:\n00: f017387 (method<PHONE_NUMBER>) -- invalid method:<PHONE_NUMBER> (22)\n (RetCode=22)")                                          |

Other information and links

There's also an interesting case where Forest fails with deserialisation error:
❯ curl -s -X POST <IP_ADDRESS>:2345/rpc/v1 \
           -H 'Content-Type: application/json' \
        --data '{"id": 4, "jsonrpc": "2.0", "method": "eth_estimateGas", "params": [{"accessList": [], "data": "0x79979f57000000000000000000000000000000000000000000000000000000000000006000000000000000000000000000000000000000000000000000000000000001a000000000000000000000000000000000000000000000000000000000000001e000000000000000000000000000000000000000000000000000000000000000a000000000000000000000000000000000000000000000000000000000001a2df43177fff0e1dcfced59ebbdb2e849e8ac9b11a91a36bcd37e78a67eb3c3e6f9e100000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000120000000000000000000000000000000000000000000000000000000000004cb2f000000000000000000000000000000000000000000000000000000000000004000000000000000000000000000000000000000000000000000000000000000010000000000000000000000003f51f953b0df96fec2d29cadf572d0480f357002000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000010000000000000000000000008ad5acb43f6644f7cef288e4ad8aa7ac76117a50000000000000000000000000000000000000000000000000000000000000000100000000000000000000000000000000000000000000000000000000000000200000000000000000000000000000000000000000000000000000000000000041c03f40cfff325ed962844dde512078f368535575ba3c13aab4f7d48ea31b51e74c7de0a59dd02777d655e6af9bff11f6a4e5c22a6eba9882a9e257465fbb94d81c00000000000000000000000000000000000000000000000000000000000000", "from": "0x8ad5acb43f6644f7cef288e4ad8aa7ac76117a50", "maxFeePerGas": "0x186a0", "maxPriorityFeePerGas": "0x186a0", "nonce": "0x25e2", "to": "0x3f51f953b0df96fec2d29cadf572d0480f357002", "type": "0x02"}]}'

{"jsonrpc":"2.0","id":4,"error":{"code":-32602,"message":"error deserializing parameter","data":{"error":"missing field `gas`","index":0,"name":"tx","type":"forest_filecoin::lotus_json::LotusJson<forest_filecoin::rpc::methods::eth::types::EthCallMessage>"}}}⏎

while Lotus fails on something else:
❯ curl -s -X POST <IP_ADDRESS>:1234/rpc/v1 \
           -H 'Content-Type: application/json' \
        --data '{"id": 4, "jsonrpc": "2.0", "method": "eth_estimateGas", "params": [{"accessList": [], "data": "0x79979f57000000000000000000000000000000000000000000000000000000000000006000000000000000000000000000000000000000000000000000000000000001a000000000000000000000000000000000000000000000000000000000000001e000000000000000000000000000000000000000000000000000000000000000a000000000000000000000000000000000000000000000000000000000001a2df43177fff0e1dcfced59ebbdb2e849e8ac9b11a91a36bcd37e78a67eb3c3e6f9e100000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000120000000000000000000000000000000000000000000000000000000000004cb2f000000000000000000000000000000000000000000000000000000000000004000000000000000000000000000000000000000000000000000000000000000010000000000000000000000003f51f953b0df96fec2d29cadf572d0480f357002000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000010000000000000000000000008ad5acb43f6644f7cef288e4ad8aa7ac76117a50000000000000000000000000000000000000000000000000000000000000000100000000000000000000000000000000000000000000000000000000000000200000000000000000000000000000000000000000000000000000000000000041c03f40cfff325ed962844dde512078f368535575ba3c13aab4f7d48ea31b51e74c7de0a59dd02777d655e6af9bff11f6a4e5c22a6eba9882a9e257465fbb94d81c00000000000000000000000000000000000000000000000000000000000000", "from": "0x8ad5acb43f6644f7cef288e4ad8aa7ac76117a50", "maxFeePerGas": "0x186a0", "maxPriorityFeePerGas": "0x186a0", "nonce": "0x25e2", "to": "0x3f51f953b0df96fec2d29cadf572d0480f357002", "type": "0x02"}]}'

{"error":{"code":1,"message":"failed to estimate gas: message execution failed: exit 33, revert reason: 0xdd88982f, vm error: message failed with backtrace:\n00: f0134698 (method<PHONE_NUMBER>) -- contract reverted (33)\n01: f0134698 (method 6) -- contract reverted (33)\n (RetCode=33)"},"id":4,"jsonrpc":"2.0"}

This might be related to the fact that this structure in Lotus gets deserialised first with some non-trivial logic behind it.


Multiselect items documentation lacks label properties

Summary
MultiSelect items attribute documentation lacks label field
Relevant information
See link:
http://react.carbondesignsystem.com/?knob-The title (title)=Section 1 title&knob-UI type (Only for `<MultiSelect>`) (type)=default&knob-Label (label)=MultiSelect Label&knob-Form validation UI content (invalidText)=Invalid Selection&selectedKind=MultiSelect&selectedStory=with initial selected items&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

If the variable array provided to the items attribute lacks a label property, the component fails.  Shouldn't this be refelected in the documentation?
items={[{id: 'item-1',text: 'Item 1', label: 'Item 1'},{id: 'item-2',text: 'Item 2', label: 'Item 2'}]}
instead of:
items={[{id: 'item-1',text: 'Item 1'},{id: 'item-2',text: 'Item 2'}]}

Hi @LongBu! 👋 The goal of the storybook story here is to show that you can have items in your array without a label field, as long as you provide the itemToString method that properly maps them.
Definitely agreed that this could be better documented! Would gladly accept your PR if you want to add it in 😄

@joshblack what's a better way of documenting this? I would love to do a PR for this issue.


baseUri should be optional for ConnectorClientOptions

Versions
NodeJS v10.15.3
BotBuilder NodeJS SDK v4.9.2
My app is written is typescript
Describe the bug
When Using customized ConnectorClientOptions for BotFrameworkAdapterSettings, the baseUri is required, but passing a value for it causes the sdk to stop working - bot is unable to send messages
To Reproduce
const botFrameworkAdapterSettings: BotFrameworkAdapterSettings = {
appId,
appPassword,
clientOptions: { baseUri: undefined, httpClient: new CustomAxiosHttpClient() }
};
const adapter: BotFrameworkAdapter = new BotFrameworkAdapter(botFrameworkAdapterSettings);
Expected behavior
The baseUri should be optional at the very least, that will allow me to omit it, and still being able to transpile my typescript app without errors

This was a bad one, thanks @EricDahlvang for fixing it.


Chaos token command does not work for Live IDs

Description
Passing in a token generated from the chaos-token command yields an improper token for Live IDs
Steps to Reproduce

Locally host the chaos extension as an Azure Function
Run the chaos token command and undergo Interactive Login
Copy the token (be sure to start copying at the beginning of Bearer and stop right before \u001b[49m\u001b[39m
Pass the token in as the accessToken flag in either chaos start or chaos stop commands

Expected Behavior
Targeted Azure resource should be enabled/disabled.
Actual Behavior
Error returned: The access token is from the wrong issuer 'https://sts.windows.net/f8cdef31-a31e-4b4a-93e4-5f571e91255a/'. It must match the tenant 'https://sts.windows.net/44c19e06-2387-4136-85e6-a8a486c5e037/'

Thinking a service principal may be able to circumvent this issue. Going to give that a try this week. Keeping this comment for bookkeeping purposes.


Update README.md

Adds schedule for public visibility to help with expectations on issues / work commitments.

:tada: This PR is included in version 3.2.7 :tada:
The release is available on:

v3.2.7
GitHub release

Your semantic-release bot :package::rocket:


Remove api-server module

Type of change

Refactoring

Description
Remove all code related to api-server as well as build system references
to it.
Moves swagger definition to documentation module for the time being, and
add configuration in documentation to replace properties in
swagger.json. Eventually this should be generated from the CRDs.
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=smoke

@lulf you can also remove creating api-server certs in systemtests OperatorManager.java and in travis, right?

@kornys this should be the case already

@enmasse-ci run tests profile=isolated testcase=CommonTest#testMessagingDuringRestartComponents

:x: Test Summary :x:
TEST_PROFILE: smoke
TEST_CASE:
TOTAL: 3
PASS: 2
FAIL: 1
SKIP: 0
:heavy_exclamation_mark: Test Failures :heavy_exclamation_mark:

testCreateDeleteAddressSpace in io.enmasse.systemtest.isolated.brokered.SmokeTest

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: isolated
TEST_CASE: CommonTest#testMessagingDuringRestartComponents
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0


Automated renders of human-readable logic

Description
Automated renders of human-readable logic

Labels

${example} --> [EXAMPLE]

Relevant

${question_1} = 'yes' --> 001 = 1
Do this for all, including complicated relevants. But those will have to be re-written manually.

Notes

Regarding possible impacts to XlsForms - It will be best to put questions in their own column. If not, we'll have to use a regexp for now

@tulvit The code for this feature will/should be implemented in the Python code. I've added you to the "assignees" list just in case you are interested in working on this feature, even though you are new at Python. If not, let me know and I will work on this one independently.


Update/Add cargo-deny

Tool information

Tool name: cargo-deny
Tool license: MIT OR Apache-2.0
Add or update? Add
Desired version: latest
Approximate size: ~10M
If this is an add request:

Brief description of tool: similar to existing Rust tool cargo-audit but with the ability to enforce licensing rules
URL for tool's homepage: https://crates.io/crates/cargo-deny
Provide a basic test case to validate the tool's functionality: cargo deny check

Area for Triage:

Question, Bug, or Feature?:
Feature.
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[x] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?
cargo install cargo-deny
Tool installation time in runtime
downloading and compiling the tool during a build takes about 10 wall clock minutes on windows-latest.
Are you willing to submit a PR?
sure!

Hi @thedataking,
given the tool popularity and its installation time, we'd rather not to add it to the images due to maintenance concerns. Please, consider using actions/cargo-deny as it seems to be pretty popular and flexible. Using actions is a recommended way to interact with images.
I am closing this issue. Please, feel free to reach out to us if you have any questions. Thank you.

@AlenaSviridenko thanks for the quick reply.  Using actions/cargo-deny is not an option here as it only supports Linux.  I don't see any indications that support for Windows is forthcoming on the actions/cargo-deny repo but it sounds like something worth exploring given the maintenance concerns you have.


Enable conformance tests for rethinkdb

Description
This makes RethinkDB a Beta component.
Note:
RethinkDB does not support ETag. As such the ETag capability has been removed from the set of returned capabilities.
Signed-off-by: Bernd Verst<EMAIL_ADDRESS>
[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

I examined the various update policies -- error, replace and update but found that these only apply to primary key collisions. There is no such concept has Etag. Perhaps this can be emulated in the future, but for the time being this state store should not advertise being an actor state store or offer ETag support.

Also FYI, transactions on multiple documents are not actually atomic for RethinkDB according to their FAQ.
Should we remove the Transaction feature also?


Android - PagerView item in horizontal ScrollView is not scrolling

Environment
"react-native": "0.71.8"
"react-native-pager-view": "^7.0.0-rc.0"
"react-native-gesture-handler": "^2.10.1"
Description
I've add PagerView inside of a horizontal ScrollView based on this example. On iOS it works well but on Android, pager is not scrolling.
import PagerView from "react-native-pager-view";
import React from "react";
import { View, StyleSheet, Animated, Dimensions } from "react-native";
import { ScrollView } from "react-native-gesture-handler";
// Also tried ScrollView from "react-native"

const { width } = Dimensions.get("window");
const HEIGHT = 300;

const AnimatedPagerView = Animated.createAnimatedComponent(PagerView);

const COLORS = ["red", "green", "blue", "yellow"];

export function ScrollablePagerViewExample() {
  return (
    <View>
      <ScrollView style={styles.flex} horizontal nestedScrollEnabled>
      {/* Also tried <ScrollView style={styles.flex} horizontal> */}
        {COLORS.map((item) => (
          <AnimatedPagerView
            key={`pager-${item}`}
            style={{ height: HEIGHT, width: width - 70, marginLeft: 40 }}
          >
            {COLORS.map((color) => (
              <View
                key={color}
                style={{
                  backgroundColor: color,
                  width: "100%",
                  height: "100%",
                }}
              />
            ))}
          </AnimatedPagerView>
        ))}
      </ScrollView>
    </View>
  );
}

const styles = StyleSheet.create({
  flex: {
    flex: 1,
  },
});

iOS - working

https://github.com/callstack/react-native-pager-view/assets/15637944/571d994d-6693-4ee3-ac5e-19d837df87e1

Android - not working

https://github.com/callstack/react-native-pager-view/assets/15637944/ad190229-f181-4257-bb4d-a886cb9f0b01
Reproducible Demo
Put above component in any component.

Some problems, any ideas ?

same issue

same issue

is it still valid?

Closing as per: https://github.com/callstack/react-native-pager-view/releases/tag/v6.6.0


Finish PR #2432: Imagenet example updates + testing

What does this PR do?

Finishes an old PR #2432 by @ruotianluo

standardize with other examples
a test that runs for 1 epoch making sure the script can run
simplifications

Hello @awaelchli! Thanks for opening this PR.

In the file pl_examples/domain_templates/imagenet.py:

Line 195:81: W504 line break after binary operator

In the file pl_examples/test_examples.py:

Line 49:80: E226 missing whitespace around arithmetic operator

Do see the Hitchhiker's guide to code style

You can still use this one... =)

It seems the current --evaluate won't load the checkpoint from --resume_from_checkpoint, right?
I saw in the doc:
# (4) test with an explicit model (will use this model and not load a checkpoint)
traiener.test(model)

oh yes, I think you are right. Would you like to send a PR?

I am not sure what is the most elegant solution:
One possibility would just be:
model.load_state_dict(torch.load(args.resume_from_checkpoint, map_location=''cpu)['state_dict'])

But I think it's too ugly.

we could simply pass the checkpoint path to .test
trainer.test(model, ckpt_path="...") # default is ckpt_path="best", but that's not available
what do you think?

https://github.com/PyTorchLightning/pytorch-lightning/blob/8eb77cd06aff18c14242e194f6959752a9b08817/pytorch_lightning/trainer/trainer.py#L538
It won't use the ckpt_path

ok, correction, then trainer.test(ckpt_path="...") without model? :)

Doesn't that assume trainer already has the model? My understanding, if you want to use this, you have to run it after trainer.fit(model) otherwise trainer doesn't know what is the model.

we could load the state at the beginning, like
if args.evaluate:
    model = ImageNetLightningModel.load_from_checkpoint(path, map_location=...)


Delete fixed-left-right-layout

This is not used anywhere.

noo, please don't delete stories! :smile:
ok, I was wrong about not getting used - could you adapt the stories to add minimal styling directly there?

noo, please don't delete stories! 😄
ok, I was wrong about not getting used - could you adapt the stories to add minimal styling directly there?

@nsams stories are back 😄