dart2js: bad optimization of "x is Null"  -- what does TypeMask.isEmpty mean?

main() {
  foo('null', null);
  foo('"x"', 'x');
  print('null is Null => ${null is Null}');
}
var foo = (s, v) {
  print('$s is Null = ${v is Null}');
};
This program prints:
null is Null = false
"x" is Null = false
null is Null => true
The problem is that 'v is Null' is replaced with 'false'.
SsaInstructionSimplifier.visitIs:
    } else if (!RuntimeTypes.hasTypeArguments(type)) {
      TypeMask expressionMask = expression.instructionType;
      TypeMask typeMask = (element == compiler.nullClass)
          ? new TypeMask.subtype(element)
          : new TypeMask.nonNullSubtype(element);
      if (expressionMask.union(typeMask, compiler) == typeMask) {
        return graph.addConstantBool(true, compiler);
      } else if (expressionMask.intersection(typeMask, compiler).isEmpty) {
                      ^^^
        return graph.addConstantBool(false, compiler);
      }
    }
expressionMask.intersection(typeMask, compiler) evaluates to a type that prints as '[null]'.
This type returns true for .isEmpty.
I would expect .isEmpty to return true only for the type representing a completely empty set of values, that does not include a null value.
However, '.isEmpty' seems to be orthogonal to '.isNullable'.
There are several places in the intersection, union and other algorithms that seem to assume that the type representing the singleton set of values {null} is also .isEmpty
So it is not trivial to strengthen .isEmpty to my expected meaning.
I think we should strength isEmpty to make  t1.intersection(t2).isEmpty a reliable test, instead of having to also check the result is non-nullable like this code from types_propagation.dart:
    TypeMask outputType = checkedType.intersection(inputType, compiler);
    if (outputType.isEmpty && !outputType.isNullable) {
If we keep isEmpty orthogonal to isNullable, TypeMask.isEmpty really needs to be documented!

Note that the test above will stop failing when I land a change that calculates isDisjoint explicitly] (I modified the line in SSA with the problem illustrated by Stephen above). However, the issue still exists in the type mask abstraction: it would be nice if isEmpty really means empty, not empty-or-null

This is fixed.


Remove open-issue-to-latest-comment from the dashboard

UNTESTED but it should work (or not work anymore 😃 )
Closes #4496
Test URLs
https://github.com/
Screenshot
What it looked like before (Sorry could not find a better screenshot)

Now

@fregante title good?

Yes the title works 👍


Intercepting state transitions

How would we go about intercepting state transitions? In the normal ui-router, you would listen for $stateChangeStart on $rootScope. Is this still the recommended approach?

You would use http://redux.js.org/docs/api/Store.html#subscribe


Allow Swagger endpoint as input

Resolves #11
I also updated the base route / to be a list of available tables if no default table is provided.
From the docs:

Alternatively, you can provide the URL to a PostGRES Swagger endpoint, such as https://www.cmi-pb.org/api/v2. This will load all tables within the endpoint to browse. If you want to save the SQLite database file that is created, you can do so with -s/--save-database (this will fail if a file already exists at the given path). Otherwise, the database will be created at .temp.db and will be deleted on exit.
sprocket https://www.cmi-pb.org/api/v2 -s database.db

I'm getting an error: jinja2.exceptions.TemplateNotFound: index.html

Sorry, fixed.

Sorry, this is not what I meant in #11. I don't want to download everything from the endpoint. I want to forward each request to the endpoint, receive the JSON response, then display that as HTML. I'll update #11 to clarify.


GCF Checksum feature

title ^^
huge thanks to @ymgve as always

I don't know how I screwed the pr up, but these commits were merged


Update readme links

@kusanele This is a pull request to main, not development. You'll have to change it  (Edit button at the top). Be very careful when you open a pull request, examine it carefully, the red flag here was that 78 files were changed...

I changed the directory of this pull request  to development.  however i am still getting the same error, not sure why.

@kusanele Have a look at this comment on formatting of code: https://github.com/ModECI/MDF/pull/358#issuecomment-1346831642

Good day @pgleeson , the dependences mentioned at #358 (comment) Were installed as per the instructions given . however i am still getting the error message about the format.
Please fins the exact thing that is causing this error attached above. Thank you

@pgleeson well , i have a new problem now. Git hub wont allow me to push the changes because its complaining about the file size.so i went on and followed some stack overflow solution that recommended that i remove the large files on my repo ,well that created more problems for me.

Please kindly assist

As we discussed in the meeting, it seems like your virtual environment folder was also added to the commit (git log --stat output). So, that's why github won't let you push---it's too big. You should only add and commit the readme file that you have modified. So, I think the simplest is to copy your readme file somewhere, create a new branch from development and paste it there and commit it to open a new pull request.
Let me know if you run into any issues---will be on slack most of the day :)

New pull request was created for the same purpose


Deprecate std.xml

See forum.

@thewilsonator is there any way you can contact @andralex or @atilaneves on this?


[New Feature]: Create, test and document a backup procedure

Checked for duplicates
Yes - I've already checked
Alternatives considered
Yes - and alternatives don't suffice
Related problems
No response
Describe the feature request
We need a reliable procedure to backup and restore the state of the SDS Elasticsearch clusters - Mozart, GRQ and possibly Metrics. Please use either the Dev Common or one of the I&T venues, checking with other developers to make sure they are not currently using it.

@LucaCinquini might be good for us to define some backup parameters / constraints.
For example, how does the following sound?

X number of days of backup maintained | where X is 14 days
Backup automatically created every X hours | where X is 12 hours
Backups must comply with https://www.elastic.co/guide/en/elasticsearch/reference/current/snapshot-restore.html

@riverma : good idea. Those parameters are ok with me. To save money, if necessary, we could also backup every 24 hours and keep the backups for 7 days.

Backup procedure documented here: https://wiki.jpl.nasa.gov/display/operasds/ElasticSearch+Backup+and+Restoration

Looks good @niarenaw - but please make sure to use the new template that @LalaP set up. See the OPERA SDS OPS PROCEDURES main page for a link to creating a template wiki page from scratch.

I also had a look, thanks for testing and documenting Nick.
May I suggest that others need to test this procedure - perhaps Lala and Sri (separately) after a successful completion of a regression test, so that the Elasticsearch indices are populated?
We also probably need to setup a cron job to backup these indices every 24 hours.

I've updated the procedure to abide by the Ops Procedure template. A nightly backup should be pretty easy to add as a cron. I think it probably makes more sense to store these in s3 rather than on mozart. Maybe we set up a new bucket and add a 14 day retention period as a lifecycle rule? or 30 days?

+1 @niarenaw to a lifecycle rule. Let's discuss the details for this.


Misinterpretation of Multiple replace Directives in Golang

What happened:
It does not apply replace directive to the module version that's also in the replace directly. For example,
replace (
     google.golang.org/grpc => google.golang.org/grpc v1.61.0
     golang.org/x/net => golang.org/x/net v0.22.0
)

If I run syft with the working dir syft ., the result will show<EMAIL_ADDRESS>is being used, which is originated from<EMAIL_ADDRESS>go.mod while both module should be overridden to the specified version.
What you expected to happen:
If I build and run syft against the binary file instead, syft <bin_file>, or go version -m <bin_file>, both will show<EMAIL_ADDRESS>is actually being used.
Steps to reproduce the issue:

Create a simple project
Import a module, let's name it module M
Use replace directive with M to any specific version
Use replace directive with another module required by M (pick one from M's go.mod), let's name it module N
cd to the project root directory, set $GOPATH with export GOPATH="$(pwd)/dep
go mod download, to download all dependencies into ./dep.
Run syft with the project directory, inspect the N's version, it will be the version specified in M's go.mod, while it should actually be the replace directive version.

Environment:

Output of syft version:

Application: syft
Version:    1.0.1
BuildDate:  2024-03-06T19:25:56Z
GitCommit:  Homebrew
GitDescription: [not provided]
Platform:   darwin/arm64
GoVersion:  go1.22.1
Compiler:   gc

OS (e.g: cat /etc/os-release or similar):

ProductName:		macOS
ProductVersion:		14.3.1
BuildVersion:		23D60

I believe this is a duplicate of https://github.com/anchore/syft/issues/2038, but it does describe a slightly different replace directive. This particular variant would be a lot easier to handle, ignoring the local filesystem references.

Hey @kzantow
I would like to work on this issue.

@bhavyastar that would be great! I've assigned you (I think that's one thing you were asking for?), let me know if there's anything you need!


fix video not rendering in markdown tables and sometimes fail to render in the chat?

i think its an issue with slashes and markdown parsing
sometimes also fail to render in chat (some prompt engineering tweak to do?)
definition of done:

"what did i do in the last 5 min? format as a table with a column showing videos of what i did" works
no more fail to render in the chat (idk how to reproduce)

/bounty 50

/attempt #191

@louis030195 i am running screenpipe currently with cli installation,so i am not able to see this thing,can you please tell me where to run this ,"is this the vercel ai chat bot or something else'  ,is there any specific proj inside examples folder that i need to run and also i think it required openai key for the chat

/attempt #191

@Mubashirshariq check instructions here https://github.com/mediar-ai/screenpipe/tree/main/examples/apps/screenpipe-app-tauri

https://github.com/mediar-ai/screenpipe/blob/main/examples/apps/screenpipe-app-tauri/components/chat-message-v2.tsx

seems fxied idk why


[BUG]

Sometimes when I enter the world I have to choose the origin again. This also happens during the gameplay, when the tps suddenly boosts or after respawning from death, pressing the 'o' for origin gui which displays "you can not have any origin". The bug have existed for a long time, Can you fix it?

Please state the version of the mod you are using, as well as the minecraft version. Please note that 1.16.5 is currently not supported.


TypeScript Template Literals break generic constraints

Bug Report
I made a SO post for it as well but this behavior feels like a bug as whether the first argument of a function is generic or not, should not disable constraints on the second.
⏯ Playground Link
Link
💻 Code
interface ChatEvents {
  connect: {
    user: string;
  };
  disconnect: {
    user: string;
    reason: "banned" | "timeout" | "leave";
  };
}

declare type ChatEvent = keyof ChatEvents;

interface SubEvents extends Record<ChatEvent, string> {
  connect: "general" | "watercooler" | "lobby";
  disconnect: "voice" | "text";
}

declare type EventWithSubEvent<T extends ChatEvent> = `${T}${
  | ""
  | `:${SubEvents[T]}`}`;

// "connect" | "connect:general" | "connect:watercooler" | "connect:lobby"
declare type ChatConnectEvent = EventWithSubEvent<"connect">;
// "disconnect" | "disconnect:voice" | "disconnect:text"
declare type ChatDisconnectEvent = EventWithSubEvent<"disconnect">;

// So far so good!

// Let's write some generics:
declare function subscribeToEvent<E extends ChatEvent>(
  event: E,
  callback: (payload: ChatEvents[E]) => void
): void;

subscribeToEvent("connect", (payload) => {
    // Correctly extracts matching type
    type TPayload = typeof payload; // { user: string; }
});

declare function subscribeToSubEvent<E extends ChatEvent>(
  event: EventWithSubEvent<E>,
  callback: (payload: ChatEvents[E]) => void
): void;

subscribeToSubEvent("connect:general", (payload) => {
    // Extracts all possible payload types instead of only the `connect` one
    /*
        {
            user: string;
        } | {
            user: string;
            reason: "banned" | "timeout" | "leave";
        }
    */
    type TPayload = typeof payload;
});

subscribeToSubEvent("connect:voice", () => {}) // Should fail but doesn't. `:voice` is a subevent of `disconnect`, not `connect`.

🙁 Actual behavior
Generic constraints stop being checked
🙂 Expected behavior
Generic constraints continue to apply

Seems like your type definition went beyond TypeScript's ability to infer type parameters.
I found a workaround: change the definition of EventWithSubEvent to the following.
declare type EventWithSubEvent<T extends ChatEvent> = T | `${T}:${SubEvents[T]}`;

Here is a little simpler reproduction:
type FooBar = "foo" | "bar"
type U<T extends FooBar> = `${T}${"" | "1"}`

declare function foo<T extends FooBar>(arg: U<T>): T;

const a: "foo" = foo("foo1")
// Actual Behavior  : foo<FooBar>("foo1")
// Expected Behavior: foo<"foo">("foo1")

Playground

This doesn't demonstrate a bug. The declaration
declare type EventWithSubEvent<T extends ChatEvent> = `${T}${
  | ""
  | `:${SubEvents[T]}`}`;

has a legal instantiation EventWithSubEvent<"connect" | "disconnect">, of which connect:voice is a legal inhabitant:
const p: EventWithSubEvent<"connect" | "disconnect"> = "connect:voice";

@RyanCavanaugh I see. Why can't it infer the T, though?


Cleaned up events.rst

Was a mix-up between TestEventSubscriber and EventTest (e.g. the definition of TestEventSubscriber referenced TestEvent::preFoo, which did not exist). To clarify this I've renamed EventTest to TestEvent.
Tried to clarify the text in the Naming Convention section.
Added note that onClear is not a lifecycle callback.
Tried to clarify the definition of Lifecycle Callbacks.
Separated key/value descriptions into XML and YAML parts since the details are different
Added note in Implementing Event Listeners section that since 2.4 you do have access to EntityManager and UnitOfWork from lifecycle callbacks.
Added example about how to use the computed changeset to modify a primitive value in preUpdate section
Added naming convention example to Entity listeners class section
The other changes are typos and small fixes.

Could we close these pull requests which are merged? It would make it easier to see all currently open and active pull requests. :+1:

Never mind my previous comment. Seems I had deleted the is:open filter. My bad.


refactor: ast node json unmarshal

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

refactor: ast node json unmarshal

Refs #405

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 36.314%

Totals

Change from base Build<PHONE_NUMBER>6:
0.01%

Covered Lines:
5442

Relevant Lines:
14986

💛  - Coveralls


Question: Current state of the project

Hey there,
I found this repo by a google search while looking if there is any python wrapper for the faceit api.
Pretty amazing work you have done! I am really looking forward to trying this out.
While looking on pypi and here I noticed there are quite a few different versions. Which one is the recommended one to use? pypi, main, dev or public?
I want to do something with the faceitapi regarding clash of clans as a game, because of that I noticed that it was missing in the Game enum, (afaik you were more interested in CSGO/CS2) so I wanted to contribute and add that by opening a pr. I would just need to know which branch I should use for that :) (and potentially any other things I stumble across). Of course all only if you appreciate a contribution from me side.
Cheers!

Hey doluk,
thank you for your kind words. Most up to date branch is the public branch, so you might want to branch from there. I appreciate any contribution so feel free to improve and add some stuff :)
Let me know if you need to know anything else.
Have fun coding!


Hash based matching of lint violations

Added a hash based approach for identifying known lint violations. This is likely not the final strategy but I want to test how it behaves on a large project.

This approach was loosley touched upon in #1


Keyboard shortcuts on wrong keys after changing from layout to other

VSCode Version: 1.52.1
OS Version: Arch Linux

Steps to Reproduce:

Use sway
Set keyboard layout to workman,us with swaymsg input type:keyboard xkb_layout workman,us
Open vscode
Keyboard shortcuts are set to keys corresponding to that layout
Change layout with Super + Space
Shortcuts persist on same keys and not changing to the keys in the current layout.
For example C key is at V in workman and V in B. Now copy is Ctrl+V and paste Ctrl+B.

Expected behavior:
Keyboard shortcuts would work with the keys set in Keyboard Shortcuts menu and change to mach the current keyboard layout.
Other knowledge:

Reloading window doesn't help. Full restart with all windows closed is the only way to get the new keyboard layout recognized
Could be fixed by using key charactes to trigger shortcuts, not keycodes which are looked up from layout at startup

Does this issue occur when all extensions are disabled?: Yes

Looking at https://github.com/microsoft/vscode/wiki/Keybinding-Issues#troubleshoot-linux-keybindings it seems that "keyboard.dispatch": "keyCode" got it working. I still think that it should be the default on Linux platforms as it's not that error prone as just using the raw keycodes is.
The configuration is showing keys by name and not their scancode and thus leads users to think it's the key names being used when in fact they're not.

Looking at https://github.com/microsoft/vscode/wiki/Keybinding-Issues#troubleshoot-linux-keybindings it seems that "keyboard.dispatch": "keyCode" got it working. I still think that it should be the default on Linux platforms as it's not that error prone as just using the raw keycodes is.
The configuration is showing keys by name and not their scancode and thus leads users to think it's the key names being used when in fact they're not.

Let's track in #24166

Let's track in #24166


Add control over header image

Please provide enough information so that others can review your pull request:
Currently, the header image for a post is bg-top, but it would be nice the control this behavior to lets say top|center|bottom, as often the interesting part of an image is in the center. Using the featured_image_class param, one can now customize the class to control the alignment of the featured header image. It is also possible to add additional classes for further customization if needed.
Explain the details for making this change. What existing problem does the pull request solve?

Test plan (required)
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.

Default case:

Same as default case but set explicitly featured_image_class = cover bg-top:

featured_image_class = cover bg-center:

featured_image_class = cover bg-bottom:

featured_image_class = contain bg-top:

Code formatting

Closing issues
Closes #729

Not sure why the build fails, it complains about something unrelated to my changes imo.

If you force-push the current state, the Netlify jobs should run without issue.


fullClasspath / runMain regression in 0.13 => 1.x

I used to be able to customise the fullClasspath of a config to add extra things onto the runMain, but that seems to have broken from 0.13 to 1.x... what should I do now?
I was using this technique in my jmh plugin https://gitlab.com/fommil/sbt-sensible/blob/master/src/main/scala/NeoJmhPlugin.scala

this seems suspicious...
  private[this] def mainBgRunTaskForConfig(c: ScopeAxis[ConfigKey]) =
    bgRun := bgRunTask(
      exportedProductJars,
      fullClasspathAsJars in (This, c, This),
      mainClass in run,
      bgCopyClasspath in bgRun,
      runner in run
    ).evaluated

and these "Jar" variants do not look at fullClasspath

ok, suitable workaround apperas to be
    dependencyClasspathAsJars in NeoJmhPlugin.JmhInternal ++= (fullClasspathAsJars in Jmh).value,

but probably not 0.13 compatible...

(btw, I closed this because I don't think it's possible to fix it at this point... I still think it's a regression)


feat(places): Add option to choose folder in places add/edit menu

re #533

Working on fixing tests now.

Live demo: http://places-folder-options-opensphere.surge.sh/
Test:

Add folders to the places tab (some nested)
Create a place from the map (right click menu > Save to Places)

verify that a folder choice is presented and defaulted to saved places
verify that the other folders are listed as options in a flattened tree

Choose a folder for the place and save it

verify that the place shows under the correct folder

Edit the place you just created

verify that the folder selected is correct

Choose a different folder and save

verify that the place moves to the correct folder

Create a place from a folder other than saved places (click the place icon to the right of the folder name in the places manager)

verify that the folder you created the place from is chosen

Save the place

verify that it saved to the correct folder

Select a folder in the places manager by clicking it
Create a place from the top of the places manager (Add Place button)

verify that the folder selected is correct (whatever folder you clicked in step 8)

Try out other combinations of creating/editing with places and folders as needed

Test looks good!


fixed context manager for multiple categorical variables in context

Issue #, if available:
In context_manager: when considering multiple categorical variables in context, appending (instead of extending) the encodings will lead to the context_values being a sequence of lists and floats. This becomes an issue when expanding the vector at
x_expanded[:, np.array(self.context_idxs).astype(int)] = self.context_values
because we are trying to assign a sequence to a list
Description of changes:
changed append to extend in emukit.core.optimization.context_manager.py and added a corresponding test to show the bug
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #305 into master will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #305      +/-   ##
==========================================
+ Coverage   89.26%   89.32%   +0.05%
==========================================
  Files         120      120
  Lines        3932     3933       +1
  Branches      454      454
==========================================
+ Hits         3510     3513       +3
+ Misses        329      328       -1
+ Partials       93       92       -1

Impacted Files
Coverage Δ

emukit/core/optimization/context_manager.py
100.00% <100.00%> (+7.40%)
: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 2db676b...4107411. Read the comment docs.


tfjob: Send event to object

Close #755
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

This change is

Coverage increased (+0.07%) to 58.061% when pulling 19da03ca128f740a415dfffbacf5b16c5d82c64d on gaocegege:event into 037e9153bac3deff2ee1f370d2bcccc633bdc98b on kubeflow:master.

/approve


tinker with shadcn

Goal:
Tinker with SE-2 x ShadCn to see if it nicely integrates with our needs.
We can go with 2 routes:

In base SE-2 replace daisyUI with shadcn
we keep base SE-2 with daisyUI but allow shadcn through extensions

The rationale behind replacing daisUI with shadcn is because on surface level I think shadcn should be able to replace daisyUI in SE-2 but sometimes as we go deep into rabbit hole we hit some bad blockers that does not go with ethos of SE-2 like #444...but yeah lets tinker and see if route 1 is an option
Even if we don't go with 1st route, I think it's really good to have shadcn as core extension in create-eth

Great!!

we keep base SE-2 with daisyUI but allow shadcn through extensions

I think it's not the case

some shadcn components conflicts with daisyui, for example sonner. I tried a week ago and it didn't work without disabling daisyui plugin
how should it work? We can't just replace daisyUI styles and logic inside all the core components and pages with shadcn. If not replace, everything will be the same daisyUI and just shadcnUi components will be added, so I can use them for creating my new components? I think it's not the case and no one will use it because all the app is still daisyUi
(less important) tailwind config with both daisy and shadcn is starting to become a mess

The rationale behind replacing daisUI with shadcn is because on surface level I think shadcn should be able to replace daisyUI in SE-2

Agree!
Btw, shadcn works fine with yarn, but their yarn copy button uses npx for yarn (looks like a bug). I used yarn dlx instead

Theme generator which can help us to generate our color https://zippystarter.com/tools/shadcn-ui-theme-generator
paste our primary color #93BBFB
But looks like we need to use it at the last steps before removing daisyui

https://github.com/scaffold-eth/scaffold-eth-2/pull/936#issuecomment-2518015531


fix(sec): upgrade golang.org/x/crypto to 0.17.0

What happened？
There are 1 security vulnerabilities found in golang.org/x/crypto v0.0.0-20220331220935-ae2d96664a29

CVE-2023-48795

What did I do？
Upgrade golang.org/x/crypto from v0.0.0-20220331220935-ae2d96664a29 to 0.17.0 for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
How can we automate the detection of these types of issues?
By using the GitHub Actions configurations provided by murphysec, we can conduct automatic code security checks in our CI pipeline.
The specification of the pull request
PR Specification from OSCS

Ping @FiloSottile


GriefDefender doesn't migrate claims from GriefPrevention on Sponge

When griefprevention-sponge is set to true in the migrator section of global.conf for GriefDefender, and the server is loaded with claims in the griefprevention directory within the plugins config directory, the migrator doesn't import them, and doesn't issue any warnings, errors, or exceptions.
It imports claim flag defaults just fine, so the setting is being seen by the plugin and acted upon.
I think GriefDefender may be written to only check the default shared mods/plugins config directory, and not to get the current plugin config directory from Sponge. This would cause this observed behavior for non-default locations of the config directory, like mine, but would work via coincidence for most server owners.
logs.zip

All GD does is copy the GriefPrevention claims folder to GD's. Just copy it manually or copy it to the standard config folder and it will migrate fine on startup.

I fixed the migrator (see PR #130), and it copies the files now, but the claims aren't present in-game. Any ideas?

Check that your world UUID(s) haven't changed, the migrator does work perfectly fine.

I combined my changes in #130 with a regenerated global.conf file, and it works now! The claims show up in-world, and the stick works to find them again.

The issue I was seeing where GD didn't migrate claims was due to hardcoded config paths fixed in my PR, and the issue where the copied claim files still didn't appear in the world was due to #132.

Marking this as resolved since PR's were merged.


MAINT: PEP-compliant code formatting

Is your feature request related to a problem? Please describe.
PyCharm throws tons of warnings but most of them seem to exist due to the code not being PEP-compliant.
That is distracting and prevents from finding actual issues in the code.
Describe the solution you'd like
Again, it is possible to reformat the files without breaking anything (manually or using an IDE). I can try and do this.
Describe alternatives you've considered
I see none other than switching the warning off.

In my opinion (which is not necessarily the opinion of other devs), either style should be enforced in CI (which it is with flake8 and perhaps to some extent mypy) or you should switch your dev environment to not be upset by parsl's CI-enforced style.
So to use a stricter style in general, I think it would be good to add enforcement of this into CI. Otherwise, that style is going to be violated after the next PR.


Implement support for multiple output destinations

Currently having more than one output in the list will raise a NotImplemented error
NotImplementedError: only single output is currently supported
I believe proper queuing needs to be implemented for this

fixed in https://github.com/20c/vaping/commit/2dd003cb0b881b7cef213ae855078f5065545ab0


fix : Information lost on repetitive clicking of SignUp button in bottom navigation bar

fixes #608

We are actually already working on this issue. We'll see which implementation is better and merge the one accordingly

@iamareebjamal I think you should merge this PR for now. If you find any better implementation you can always put an issue for this later.

That's counter productive> Why would we merge something just to revert or replace it later?

@simarsingh24 You had the fix ready for retaining the fragments, right? Can you please send a PR

@shikhart98 is the issue still active after the last PR that got merged (retaining fragments). If yes please share a gif of the same and also update the PR otherwise please close this. Thanks

I'll send it by tonight!

@simarsingh24  I tried running the the app, the problem is still there.

@shikhart98 could you please attach a gif for the same

@simarsingh24
I think it is not yet solved.

@simarsingh24 any update on this?

@shikhart98 see why error or toast are not displayed. please proceed

The problem is that the fix applied by @simarsingh24 was just in MainActivity, follow the same approach in AuthActivity

Can I do it?

Yes


Answer and Decline buttons Issue

i am using agora and i create my own screen for 2 users start the call or video call, and i use call kit for display notification,
when the app is opened and notification received no problems and when the user click on answer it navigate him to answer screen, but when the app totally closed  when the the user click on answer button its not navigate to the answer where the other user is waiting its only opens the app any solution for this please

please check https://github.com/hiennguyen92/flutter_callkit_incoming/blob/be650a7701fd2c6ad64996cd243bd2ed98a6f5b5/example/lib/main.dart#L101

var currentCall = await getCurrentCall(); get null data i tried it before

I need your help i use your plugin to view incoming calls, my problem is when the user receive background  notification when app is terminated, i can't navigate to answer screen when click answer, and if you have another way to open the answer screen give it to me


Setting up Kanban board for the Ruby Capstone Project - My Catalog

Project link
Team Members
Team member #1 Asha S V
Team member #2 Joshua Blue-Jack
Team member #3 Agasha Bornittah

Status: Approved :heavy_check_mark:
Hi Team,
Your task is complete! 🍾🍾 Congratulations!  🎉💯🌟
Highlights 👍

Board has required number of tasks
Fair distribution of tasks

Keep soaring higher! 💻 🍷 🚀
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Web Address Readme.md

Pull Request Description Here
Adresses Issue (start typing # to see a list of issues)
Before you submit, remember to:

[ ] Pull in all changes from main to reduce conflicts
[ ] Request reviews from @earthlab-education/ea-students

I went ahead and merged this @sear2701 and @akpost21


docs: add basic usage guide to sphinx docs

I should probably add a one-sentence section about installation: pip install psi-j-python. Since that's all there is to it, right?

I should probably add a one-sentence section about installation: pip install psi-j-python. Since that's all there is to it, right?

Not quite - the module is not yet pushed to pypi.  I would actually suggest to begin doing though, even before we get to 1.0 - it simplifies testing and integration, a lot.


cmd/compile: real/imag builtins mishandle untyped arguments

Go spec says:

For real and imag, the argument must be of complex type, and the return type is the corresponding floating-point type: float32 for a complex64 argument, and float64 for a complex128 argument. If the argument evaluates to an untyped constant, it must be a number, and the return value of the function is an untyped floating-point constant.

$ cat /tmp/cplx.go
package cplx

const (
        a int = real(0i)
        b = imag(0)
)
$ go tool compile /tmp/cplx.go
/tmp/cplx.go:4: cannot use real(0i) (type float64) as type int in const initializer
/tmp/cplx.go:5: invalid argument 0 (type int) for imag
$ gccgo-4.8 -c /tmp/cplx.go
$ gotype /tmp/cplx.go

Duplicate of #11945. Closing.


Add alt RPC and tweak error handling

Based on a follow up discussion with @KyleMoser this PR does not address the root of the problem.  An alternate RPC server will not have this data.  Closing this PR


merge upstream

i need to test it before merge

hm wait

wrong

wrong, i will open a new PR later


Get union of all possible return types of a function

For example:
type F = () => number | string
type G = returnof F // number | string

If there is interest, I will write up a proposal.
Note that this is distinct from https://github.com/Microsoft/TypeScript/issues/6606, which gives a return type given parameter types.

I think the example should be
type F = {
    (): number;
    (): string;
}

You're right - your example shows more clearly that we're taking the union of all return types. In this example, returnof both my F and yours should give number | string.

This is now done with ReturnType in https://github.com/Microsoft/TypeScript/pull/21847.


Replace rental with ouroboros

rental is not maintained any more and users
are encouraged to switch to alternatives.

closing in favour of #46. Hadn't noticed it lol.

Feel free to reopen. We haven't decided on the preferred way forward yet.

What a curious coincidence :D

Thanks for this. It was merged in manually as 7b51ea85e27004b55ebdc86e60da24f149503fb5.


[jlink] add build option RTT_BUFFER_SIZE to set the RTT buffer size

When debugging OT via the RTT, OT output logs are often truncated because the RTT buffer size is too small. This commit adds the build option RTT_BUFFER_SIZE to easily change the RTT buffer size.

Thanks. What's the buffer size you found suitable?

Generally, if the chip have enough RAM, I would set it to 2048.


[WIP] linting rules wording

Spelling out some linting rules and setup.

will go through all and also add good/bad examples

ups regarding styl.scss :(
linked to yaml and created headings from the list

rake aborted!
No Rakefile found (looking for: rakefile, Rakefile, rakefile.rb, Rakefile.rb)

Should this be using govuk-lint gem?

don't think we need ruby so sass-lint, as it's node based

@igloosi First line reads "The following scss-lint", which led me to believe that's what's being used.

oh, the comments in the sass-lint file


chore(footer): add Status link to Oak footer and tidy up links

Adding status link and use first party domain for careers to direct to jobs page.
Description

Added Status to bottom of Oak section in footer that links to https://status.thenational.academy
Edited Careers link to direct to https://jobs.thenational.academy instead of direct link

How to test

Go to homepage
Scroll to footer
You should see Status at the bottom of the Oak section (middle column on desktop)
Click link and it should take you to https://status.thenational.academy
You should see Careers third in the list of the Oak section (middle column on desktop)
Click link and it should take you to https://jobs.thenational.academy and redirect to BeApplied

Screenshots
How it used to look:

How it should now look:

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner

:tada: This PR is included in version 1.183.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cannot Found new Controller for second theme

I have create other theme
but Target class [Themes\second\controllers\HomeController] does not exist
route
Route::middleware(['web'])->group(static function () {
    Route::namespace(theme_namespace())->name('theme/')->group(static function () {
        Route::get('/','HomeController@index')->name('index');
    });
});

try to add this to your composer.json
    "autoload": {
        "psr-4": {
            "App\\": "app/",
            "Database\\Factories\\": "database/factories/",
            "Database\\Seeders\\": "database/seeders/",
            "Themes\\": "resources/views/themes/"
        }
    },

i add this to the docs, thanks

Thanks that problem solve, but every time I add new route that always call "php artisan optimize" how to make it works auto without command ^_^


Appstudio update test-component-pac-xcyy

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-xcyy-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
22 seconds
clone-repository

✅ Succeeded
21 seconds
build-container

✅ Succeeded
14 seconds
inspect-image

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
34 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
15 seconds
label-check

✅ Succeeded
15 seconds
optional-label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-xcyy-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
22 seconds
clone-repository

✅ Succeeded
21 seconds
build-container

✅ Succeeded
14 seconds
inspect-image

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
34 seconds
clamav-scan

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
15 seconds
optional-label-check

✅ Succeeded
15 seconds
label-check

✅ Succeeded
6 seconds
show-summary


⚠ AGEG website (redirect) has degraded performance

In 52d34cd, AGEG website (redirect) (https://ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2787 ms

Resolved: AGEG website (redirect) performance has improved in 8084f7d.


[zh-cn]sync blog 2022-09-07-iptables-chains.md

[zh-cn]sync blog 2022-09-07-iptables-chains.md

Sorry for the failure it's because of my changes. If you want it to work:

Synchronize your fork with upstream
Get the new changes on main: git fetch && git checkout main && git pull origin main
Rebase your branch on main git checkout 2022-09-07-iptables-chains.md && git rebase -i main

/approve


Update Post “is-my-house-safe-from-termites-if-it-is-made-from-hardwood”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
71a9298f62da6572c06d39b2c3bd114671fa1447

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/652ccd9bed36800008a26e50


Check-in pubspec.lock

So we can track as the example is updated – since it’s not published as a package

While we are at it, do you think it would be worthwhile to update the dependency on angular2 so that we limit to the current minor version like we did with the components?

Checking in the lock file kinda does that – folks can see if they change the dependencies that it was checked-in with.

PTAL?


chore: align code solution with explanation

code solution aligned with route groups explanation
https://github.com/sveltejs/learn.svelte.dev/blob/main/content/tutorial/04-advanced-sveltekit/04-advanced-routing/04-route-groups/README.md

Thank you, but #475 contained the more desired solution, so closing in favor of that.


Pixi offline upgrading and everything else

Problem description
Hi! Pixi is a great tool, I'm enjoing using it on my personal laptops and other.
But using pixi for serious commercial projects with cybersec limitations is a huge pain. For example, I have disconnected all my GPU clusters from internet access and setted up mirrors and cachers: Sonatype Nexus, Harbor, etc.; so I can work in a controlled environment. And other tools, like docker, podman, pdm, poetry, pip, conda/mamba is ready and can be upgraded with apt/dnf/pipx - pixi is not (self-upgrade requires internet access and https://github.com/prefix-dev/pixi/issues/1764)
I see only two ways to fix this:

Write and open-source your own Nexus-like tool for mirror and cache packages (maybe like panamax for Rust crates)
Make pixi offline-friendly and also distribute it with apt/dnf/pipx/etc.

Anyways, thank you for your work! Great tool!

getting pixi into apt and dnf would be great 👍🏻 not sure how the rules for these packaging ecosystems are, though. you can definitely maintain your own packages but it would be better if we managed to get them upstream into the main repos

You can setup a selfhosted conda package mirror with Quetz
I'd be open to get pixi on pypi but currently don't have the capacity. apt or dnf would be even harder for me.
That said contributions are always welcome! So if anyone has the time and knowledge please help us get pixi distributed though other ecosystems.

You can setup a selfhosted conda package mirror with Quetz

I have a problem with this. You can check my other opened issue, that I've linked in topic message, I've explained everything there.

I have a problem with this. You can check my other opened issue, that I've linked in topic message, I've explained everything there.

This issue in #1764  is unrelated to the self-hosted conda package mirror. Let me explain that issue in #1764.

@toiletsandpaper would you also require like an --offline mode, where it does not touch the internet during pixi operations as well?

What kind of operation requires internet connection? I don't really think that tools like pixi really should require internet at all. Maybe for self-update or initial installation, but nothing else.
I saw pixi trying to dowload pypi-mappings from github. If you want to keep it up to date even if user don't want to upgrade pixi - yeah, --offline mode that forces pixi to disable mapping updating would be great. Maybe even as a flag in .config folder?

Well, a lot of people like the option of having up to date packages and index data by default 😄 This is how most package managers operate. However, thats why uv and other tools offer a --offline flag.

Well, a lot of people like the option of having up to date packages and index data by default 😄 This is how most package managers operate. However, thats why uv and other tools offer a --offline flag.

Yeah, I get it, but I have pypi and conda mirrors and --offline will ignore my mirrors too, right? If so, then --offline is not that relevant in my case, idk about others :)

If you only have conda dependencies you can use pixi-pack which is the equivalent of conda-pack.

I do really hope that someone with good rust expirience can make use of https://github.com/PyO3/maturin that was built to publish rust tools on PyPI. The closes example is https://github.com/astral-sh/uv as it can be downloaded via pip (pipx install uv)


Add France Holidays

Could it be possible to include France national holidays?

I made a pull request (#48) for that, feel free to contribute.

Hello, in which milestone this feature will be available ? please


grpc: 1.39.1 -> 1.40.0

Motivation for this change
https://github.com/grpc/grpc/releases/tag/v1.40.0
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This update has broken libtensorflow and sysdig (and their dependents: https://hydra.nixos.org/eval/1702650#tabs-now-fail); the apparent cause is that grpc now generates code that has to be linked to libabsl_synchronization, but users of the generated code do not link to it.

Sounds like it needs to be added to propagatedBuildInputs.

@orivej sorry about that, it should be fixed by https://github.com/NixOS/nixpkgs/pull/137456


Fix missing Some in deployment transaction hash

Serves me right for not making sure that an example uses this.

Only broken if the feature was used.

We can release both together as 0.12.0. When your PR is ready, you or I can merge both, and bump the version.

I suggest bumping to 0.11.1, but not a strong opinion either way.


Create a PR check workflow

Summary

Create a PR check workflow

Deadline

09/22/2023

Estimated Work Time (in hrs)

1 hrs

Related commits:

Create PR check workflow

Summary

First PR was performed.
Testing was a success.

Linked Issue: #40

Summary

Created other PR workflows (GitHub Actions)
-- Create Branch from Issue
--
Improvised PR check workflow

Estimated Work Time (in hrs)

12 hrs

Related commits:

Summary

Improvised PR check workflow

Related issues:

#9


Provide option on store filtering to filter based on modified values

Currently store filtering works off of committed (unmodified) values only. This is to avoid confusing behavior such as viewing a filtered list of records in a grid, editing one such that it no longer matches the filter, and having that record immediately disappear from view.
However it can be equally confusing in the other direction, where the user edits a record and now expects it to pass/not pass a filter, but does not see that happening and is unsure if their edit "worked."
If we provided the developer with an option to filter based on committed vs. modified, it would allow them to adjust based on the particular use case, and would generally make this behavior more clear / documented.

@haynesjm42 - doing our weekly ticket triage and still hoping to get this into a release at the end of next week - possible for you to take a look?

Sure will do

Just encountered this myself when trying to apply filters to a grid with exclusively added records (always starts from an empty state in my case).  Was able to work around the issue with the following code, but a flag as described above would certainly be useful here:
private addAction(): RecordActionSpec {
    return {
        icon: Icon.add(),
        intent: 'success',
        text: 'Add',
        actionFn: ({gridModel}) => {
            const id = XH.genId();
            gridModel.store.updateData({add: [{id}]});
            // Add committed record to the store first to allow filtering
            gridModel.store.modifyRecords({id, isModified: true});
            gridModel.beginEditAsync({record: id});
        }
    };
}

We had a run at this in #2982 but did not reach a satisfactory solution - see PR for much more context. Closing (until it comes up again)

This came up again today in a client-app.  That app was loading several "uncommitted" records by calling Store.addRecords in order to enable validation, since our RecordValidators only run on uncommitted records.  However, it also expected those records to respect filters.  For now, that app is going to disable filtering for this grid (validation is more important than filtering in this case), but ideally we could come up with a solution in Hoist.  A couple options that have been floated:

Add Store.allowFilteringAddedRecords - could be checked by FieldFilter.getTestFn to determine whether to pass adds
Add Record.isFilterable to specify at the record-level. This is more complex, and unsure where we'd expect developers to set this.

Related but different issue - only FieldFilter (via its getTestFn) automatically passes adds. FunctionFilter does not unless a developer specifies that behavior via its testFn.  We should probably standardize this by having FunctionFilter.getTestFn check if a candidate is an add before delegating to its testFn.

@minusleaf and I were bit again by this today - we were counting on being able to filter a store that we also wanted to use precisely for its added/modified record tracking.


Develop util peturb

Added utils.sample_perturb that allows for under/over sampling specific Crime_type (only works on broad crime_types) in reference to issue #7

Merging current version of under/oversampling in. Does not fully close issue #7 as we want to be able to over/under sample crime_Descriptions too


Select not working

Hey coniel,
first off, thanks a lot for making this package. I'm coming from blaze to react and this form lib is really useful! I have one problem however, which is that the select box is not working.
Consider the following component:
NewProjectForm = React.createClass({

    render() {
        return (
            <Form schema={this.getSchema()} id="registration-form" onSubmit={this._onSubmit}>
                <TextInput name="projectName" layoutStyle="first-half"/>
                <Select name="databaseType" options={[{label: 'None', value: 'none'}, {label: 'Oracle', value: 'oracle'}]} />
                <SubmitButton label="Create Project" />
            </Form>
        )
    },

    getSchema() {
        return new SimpleSchema({
            projectName: {
                type: String,
                max: 60,
                placeholder: "My testable project"
            },
            databaseType: {
                type: String,
                allowedValues: [],
                placeholder: 'My database type'
            }
        });
    },
    _onSubmit() {

    }
});

It renders the field, dropdown button and label correctly. However, no items seem to get rendered. I have attached a screenshot of the result.

Hey there again,
I got it fixed. It would be nice to include in the README.md that you need add your<EMAIL_ADDRESS>to execute this code when you bootstrap your meteor application:
injectTapEventPlugin();

Without this, it does NOT work out of the box.

Hey, thanks for opening an issue, glad you like the package! I also struggled with the exact same issue you had for a while, I had planned to add that to the readme, but totally forgot. I'll add it now, thanks for the reminder!


Font-Face and Base64 Data-URI

@projectmushroom please review.
There is currently FOUC and FOUT however it's very small. There is part 2 to fix this.

Looking to test the difference in render blocking Font currently.

mmm I don't know about this one, the encoded inline fonts are heavy, the complied css is 1.2mb, the Baton fonts with their respected .eot .woff .woff2, all 9 of them are 594KB combined, and the browser won't even load all of them, just the first one supported, we can also probably get way with just using .woff and woff2, its widely supported nowadays and out feature-phones version uses the fallback anyway


Fix README

Because double-backslash is not escaped properly, the line "\n" to "\\n" displays "\n" to "\n" in Markdown notation.
In addition, awk command seems like to be incorrect.

Thank you for the fix.
Let me check it.

I noticed the command $ awk '{gsub("\\\\n","\\\\n"); print $0;}' doesn't work on fish.
On bash or zsh , the command works.
I'd like to close this PR.
I'm sorry for bother you.

I see.
We haven't tested any commands on fish unfortunately.
But, thank you for the report.


Description for IP Address Whitelist/Blacklist

It would be nice to have a description field for IP Address Whitelist/Blacklist

Yes, please add this future!
It is very hard to remember the multiple IP addresses in the allowlist.


Add recursion limit in eval of conditions

There already is a limit during parsing for a condition.
However, it is still possible to generate an expression that is very deep, which would
trigger a stack overflow on evaluation, even without the parsing limit being reached.
Add a limit on this, test it, and make it configurable.

Codecov Report

Merging #4 (6ef97f2) into master (0f2f8fb) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   94.25%   94.26%
=======================================
  Files          55       56    +1
  Lines       17119    17146   +27
=======================================
+ Hits        16135    16162   +27
  Misses        984      984

Impacted Files
Coverage Δ

boreal/src/compiler/error.rs
100.00% <100.00%> (ø)

boreal/src/compiler/expression.rs
100.00% <100.00%> (ø)

boreal/src/compiler/mod.rs
97.15% <100.00%> (+0.03%)
:arrow_up:

boreal/src/compiler/params.rs
100.00% <100.00%> (ø)

boreal/src/compiler/rule.rs
100.00% <100.00%> (ø)

boreal/src/compiler/tests.rs
96.71% <100.00%> (+0.01%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


false positive resource of type javax.net.ssl.HttpsURLConnection

Please make sure your issue is not addressed in the FAQ.
Please include the following information:

[x] The version of infer from infer --version.
Infer v1.1.0
[x] Your operating system and version, for example "Debian 9", "MacOS High Sierra", whether you are using Docker, etc.
Ubuntu 20.04
[x] Which command you ran, for example infer -- make.
infer -- javac javacheck/HttpURLConnectionTest.java
[x] The full output in a paste, for instance a gist.
Please attention, ;the javax.net.ssl.HttpsURLConnection extends java.net.HttpURLConnection extends java.net.URLConnection, all they itself have no close() method.

javacheck/HttpURLConnectionTest.java:18: error: Resource Leak
resource of type javax.net.ssl.HttpsURLConnection acquired by call to openConnection() at line 13 is not released after line 18.
16.               return conn;
17.           } else {
18. >             System.out.println("url open connection is not HttpURLConnection");
19.               // Logger.error(TAG, "url open connection is not HttpURLConnection");
20.               throw new IOException();
Found 1 issue
Issue Type(ISSUED_TYPE_ID): #
Resource Leak(RESOURCE_LEAK): 1

[x] If possible, a minimal example to reproduce your problem (for instance, some code where
infer reports incorrectly, together with the way you run infer to reproduce the incorrect
report).

package javacheck;

// import androidx.annotation.NonNull;

import java.io.IOException;
import java.net.HttpURLConnection;
import java.net.URL;
import java.net.URLConnection;

class HttpURLConnectionTest {

    protected HttpURLConnection connect_FP(URL url) throws IOException {
        URLConnection urlConn = url.openConnection();
        if (urlConn instanceof HttpURLConnection) {
            HttpURLConnection conn = (HttpURLConnection) urlConn;
            return conn;
        } else {
            System.out.println("url open connection is not HttpURLConnection");
            // Logger.error(TAG, "url open connection is not HttpURLConnection");
            throw new IOException();
        }
    }
}

I also encountered the same problem

okay, nice to meet you here.


config-linux: specify the default devices/filesystems available

Fixes #95
Signed-off-by: Brandon Philips<EMAIL_ADDRESS>

I'm still in favor of just mounting devtmpfs for the devices 2.
That's currently broken in runC (opencontainers/runc#243), but the
spec shouldn't block based on runtime implementations.  The only
devtmpfs pushback I've heard is that you don't want the mounted
devices to depend on optional kernel features [2,3], but it seems like
the two sticky cases would be things like:
a. The kernel doesn't support tmpfs, /dev/random, or some other
feature this PR would require.
b. The kernel suppoers devices (like /dev/comedi0) that this PR
doesn't require.
For (a), I'd rather allow those systems to use OCI bundles, even
though they have a very limited kernel.  Does that sound ok to you?
For (b), I'd rather get those devices auto-mounted via devtmpfs.  You
can always use cgroups to blacklist/whitelist devices you don't want
the container accessing, so I don't see a problem with exposing them
via devtmpfs.  Does that sound ok to you?

LGTM

LGTM

LGTM

On Wed, Sep 09, 2015 at 10:16:49AM -0700, Mrunal Patel wrote:

Merged #164.

Just to document the devtmpfs resolution, in today's call we decided
that:

This is a required lower bar.  So trying to run an OCI runtime on
kernels like (a) in my previous post without /dev/random 1 will
result in errors.

Runtimes will only create devices if the listed entry is missing.
That way we don't clobber entries in the host's filesystem if the
user launches a container without a mount namespace.

Runtimes will create the listed devices before iterating through
config-specified mounts.  So you can bind-mount the required entries
to other locations via your config-specified mounts (although I'm
not sure why you would want to do that ;).  This last point didn't
get a lot of airtime in the call, so it may not be settled.


[BUG] Open Stage Control module - prevent disconnects

Is it best to connect via TCP rather than UDP (set TCP PORT to 53001 and TCP TARGETS to qlabIP:53000), or to still connect via UDP and provide a heartbeat (/workspace/id/thump) every minute?

Given option for either. Should ask further opinions of what is better.
Currently, assuming TCP is best for reliability.

Try to update documentation about this rather than leaving it as an issue.

Further research needed as to which is actually the better option


Eddystone

Hi,
Is there any implementation of Eddystone beacon ?
Tks,

Hi @medrockstar,
Unfortunately not at this moment. The Eddystone beacon is on the product's roadmap and should be available within next few months.

Hi @jkurdel !
Is the Eddystone support still in the roadmap? Is it already implemented?
Thanks!


'i2s_pop_sample' was not declared in this scope

how to solve?
I've tried to use 1.0.6 ESP32 board instead 2.0.6 and the error is
" exit status 1
Error compiling for board ESP32 Dev Module."

Try an older version. 1.0.5 or 1.0.4 of ESP32 library.

I tried without success 1.0.5, 1.0.4, 1.0.3

Did you try to clean your project and restart the visual studio or the Arduino IDE? By the way which one are you using VS or Arduino IDE?

Arduino IDE

Try to uninstall the ESP32 board manager and install the 1.0.3 and create a new project add the source code files to it.

Hello,
I ran into the same problem with an ESP32S3 zero, Arduino IDE 2.3.3 and esp32 board manager v3.0.7 (based on esp-idf 5.1). Do not use esp32 v3.1.x, as there's something broken at least in connection with the INMP441.
After a considerable amount of time in the google hell of random source code, I could now piece together a solution (at least for me, I don't use LEDMatrix or bluetooth, my usage of this code stops at the output of fftResult):

Include the suggested changes from here to replace i2s_pop_sample and i2s_read_bytes with the new i2s_read function: https://github.com/s-marley/ESP32-INMP441-Matrix-VU/issues/5#issuecomment-950307064
After i2s_driver_install and i2s_set_pin, now I2S needs to be started explicitly like so (taken from here):
err = i2s_start(I2S_PORT);  if (err != ESP_OK) {      Serial.printf("Failed starting i2s: %d\n", err);      while (true);  }  Serial.println("I2S started.");

Maybe this will be of help to some of you. I do not have an in-depth explanation for this besides "new library, new problems", as those gory details are beyond my knowledge / skill.
Thank you for your code / repository, @jozsefcsiza !


Support score type threshold in radial search

Description
This PR allows user to use score as threshold to search vectors, the search result contains all docs which score higher than the score for Lucene and Faiss engines ANN search.
Usage
During index mapping and indexing stages no behavior change. Some query examples:

Faiss engine with L2 space type

curl --location --request GET 'http://localhost:9200/target-index-faiss/_search' \
--header 'Content-Type: application/json' \
--data '{
  "size":12,
  "query": {
    "knn": {
      "my_vector1": {
        "vector": [7.5, 8.5],
        "score": 0.5
      }
    }
  }
}'

// Query results
{
    "took": 4,
    "timed_out": false,
    "_shards": {
        "total": 1,
        "successful": 1,
        "skipped": 0,
        "failed": 0
    },
    "hits": {
        "total": {
            "value": 1,
            "relation": "eq"
        },
        "max_score": 0.7462686,
        "hits": [
            {
                "_index": "target-index-faiss",
                "_id": "1",
                "_score": 0.7462686,
                "_source": {
                    "my_vector1": [
                        7.0,
                        8.2
                    ],
                    "price": 4.4
                }
            }
        ]
    }
}

Lucene engine with Inner product space type

curl --location --request GET 'http://localhost:9200/target-index-lucene/_search' \
--header 'Content-Type: application/json' \
--data '{
  "size":12,
  "query": {
    "knn": {
      "my_vector1": {
        "vector": [7.0, 8.5],
        "score": 25
      }
    }
  }
}'

// Query results
{
    "took": 303,
    "timed_out": false,
    "_shards": {
        "total": 3,
        "successful": 3,
        "skipped": 0,
        "failed": 0
    },
    "hits": {
        "total": {
            "value": 4,
            "relation": "eq"
        },
        "max_score": 69.45,
        "hits": [
            {
                "_index": "target-index-lucene",
                "_id": "13",
                "_score": 69.45,
                "_source": {
                    "my_vector1": [
                        -2.0,
                        9.7
                    ],
                    "price": 18.0
                }
            },
            {
                "_index": "target-index-lucene",
                "_id": "17",
                "_score": 39.3,
                "_source": {
                    "my_vector1": [
                        -5.7,
                        9.2
                    ],
                    "price": 3.1
                }
            },
            {
                "_index": "target-index-lucene",
                "_id": "7",
                "_score": 34.45,
                "_source": {
                    "my_vector1": [
                        -1.9,
                        5.5
                    ],
                    "price": 11.0
                }
            },
            {
                "_index": "target-index-lucene",
                "_id": "3",
                "_score": 27.5,
                "_source": {
                    "my_vector1": [
                        -3.5,
                        6.0
                    ],
                    "price": 19.1
                }
            }
        ]
    }
}

Issues Resolved
Part of https://github.com/opensearch-project/k-NN/issues/814
Check List

[x] Commits are signed as 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.

Minor comments overall code looks good to me.

In general, Im not in favor of "score" naming. Users are going to expect it to match the final score produced by the search results, regardless of the query type. So, they will be confused and cut issues when a result is returned from a more complex query with a hit score that is below the score they passed in the filter. I prefer the name "similarity" - this is what lucene refers to this as - https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/search/FloatVectorSimilarityQuery.java - it will disambiguate this case from the user experience.

In general, Im not in favor of "score" naming. Users are going to expect it to match the final score produced by the search results, regardless of the query type. So, they will be confused and cut issues when a result is returned from a more complex query with a hit score that is below the score they passed in the filter. I prefer the name "similarity" - this is what lucene refers to this as - https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/search/FloatVectorSimilarityQuery.java - it will disambiguate this case from the user experience.

I also thought about similarity. But I think user can easily understand the value mean of naming score in k-NN. The name similarity might be confused especially for feature new users for what value need provide. When say similarity, it could be related to score similarity or distance similarity. In Lucene, it only have score threshold, so I think it's fine to use similarity naming, in OpenSearch k-KNN we support both score threshold and distance threshold, so I think score as lower level words might be more suitable for us?
cc: @vamshin @navneet1v @dylan-tong-aws


Buying items sold to a NPC reduces their value

Description
Selling an item to a NPC then buying it back reduces its value as if the item was one of the items the NPC sells normally. Doing this over and over reduces the value until it becomes 1.
Steps to Reproduce

Sell any item to a NPC
Buy the item back

Expected behavior
The item's value is the same as it was before being sold
Actual behavior
The item's value is 1/5 the value the item had before being sold
Reproduction frequency
Always
Versions
tModLoader v<IP_ADDRESS>
Mods
None

Actually poor fix, noticed by another dev.


Sodium errors in Raspberry Pi 4 running latest Raspbian

I'm getting errors when running sudo ./chipper/start.sh. Looks like an issue with the Sodium Go wrapper
/root/go/pkg/mod/github.com/jamesruan/sodium@v0.0.0-20181216154042-9620b83ffeae/generichash.go:56:72: cannot use _Ctype_ulong(cryptoGenericHashStateBytes) (value of type _Ctype_ulong) as type _Ctype_uint in argument to (_Cfunc_sodium_malloc)
/root/go/pkg/mod/github.com/jamesruan/sodium@v0.0.0-20181216154042-9620b83ffeae/generichash.go:71:72: cannot use _Ctype_ulong(cryptoGenericHashStateBytes) (value of type _Ctype_ulong) as type _Ctype_uint in argument to (_Cfunc_sodium_malloc)
/root/go/pkg/mod/github.com/jamesruan/sodium@v0.0.0-20181216154042-9620b83ffeae/generichash.go:99:74: cannot use _Ctype_ulong(cryptoGenericHashStateBytes) (value of type _Ctype_ulong) as type _Ctype_uint in argument to (_Cfunc_sodium_malloc)
/root/go/pkg/mod/github.com/jamesruan/sodium@v0.0.0-20181216154042-9620b83ffeae/sign.go:226:67: cannot use _Ctype_ulong(cryptoSignStateBytes) (value of type _Ctype_ulong) as type _Ctype_uint in argument to (_Cfunc_sodium_malloc)

Device is a genuine Raspberry Pi 4 with 8Gb (bought from the official Raspberry Pi store) running Raspbian Bullseye.
PRETTY_NAME="Raspbian GNU/Linux 11 (bullseye)"
NAME="Raspbian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

I'm a software engineer, but have never used Go, so not entirely sure what I'm looking at here. Looks like a typing issue. I note this line in go.mod, which I'm guessing is the offending module:
github.com/jamesruan/sodium v0.0.0-20181216154042-9620b83ffeae // indirect

How to proceed please?

Ran go get github.com/jamesruan/sodium in chipper which has upgraded it to 1.0.14 and it seems to be running now. Will test. Would you like a PR?

Seems to be working alright now with the upgraded sodium module.

Is the OS 32-bit or 64-bit? A PR would be appreciated.

PR is here: https://github.com/kercre123/wire-pod/pull/111
Seems to work on my machine. A fresh install worked and is driving my vector.
pi@escapepod:~/wire-pod $ uname -m
armv7l

So I think that's 32 bit.


terraform-ls: 0.33.1 -> 0.34.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/terraform-ls/versions.
meta.description for terraform-ls is: Terraform Language Server (official)
meta.homepage for terraform-ls is: https://github.com/hashicorp/terraform-ls
meta.changelog for terraform-ls is: https://github.com/hashicorp/terraform-ls/blob/v0.33.3/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.33.3 with grep in /nix/store/ynx80amdp6i5hzwwxpm6fy8fn8vp50jy-terraform-ls-0.33.3
found 0.33.3 in filename of file in /nix/store/ynx80amdp6i5hzwwxpm6fy8fn8vp50jy-terraform-ls-0.33.3

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

terraform-ls
vscode-extensions.hashicorp.terraform

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ynx80amdp6i5hzwwxpm6fy8fn8vp50jy-terraform-ls-0.33.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A terraform-ls https://github.com/r-ryantm/nixpkgs/archive/0e9795e727000c1dbc2709bd1a2290398e16a91e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0e9795e727000c1dbc2709bd1a2290398e16a91e#terraform-ls

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ynx80amdp6i5hzwwxpm6fy8fn8vp50jy-terraform-ls-0.33.3
ls -la /nix/store/ynx80amdp6i5hzwwxpm6fy8fn8vp50jy-terraform-ls-0.33.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

terraform-ls
vscode-extensions.hashicorp.terraform

Maintainer pings
cc @martinbaillie @06kellyjac for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/terraform-ls/versions.
meta.description for terraform-ls is: Terraform Language Server (official)
meta.homepage for terraform-ls is: https://github.com/hashicorp/terraform-ls
meta.changelog for terraform-ls is: https://github.com/hashicorp/terraform-ls/blob/v0.34.1/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.34.1 with grep in /nix/store/89nziac9kwdg9gsbd5wkghhdf5ilmqip-terraform-ls-0.34.1
found 0.34.1 in filename of file in /nix/store/89nziac9kwdg9gsbd5wkghhdf5ilmqip-terraform-ls-0.34.1

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

terraform-ls
vscode-extensions.hashicorp.terraform

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/89nziac9kwdg9gsbd5wkghhdf5ilmqip-terraform-ls-0.34.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A terraform-ls https://github.com/r-ryantm/nixpkgs/archive/9273e9898db15315202bb33376c70c64c1b82f10.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9273e9898db15315202bb33376c70c64c1b82f10#terraform-ls

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/89nziac9kwdg9gsbd5wkghhdf5ilmqip-terraform-ls-0.34.1
ls -la /nix/store/89nziac9kwdg9gsbd5wkghhdf5ilmqip-terraform-ls-0.34.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

terraform-ls
vscode-extensions.hashicorp.terraform

Maintainer pings
cc @martinbaillie @06kellyjac for testing.


Allow querying annotations by region.

Annotation elements can be restricted by location, details, size, or the standard paging controls.

Codecov Report

Merging #180 into master will increase coverage by 0.96%.
The diff coverage is 97.84%.

@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
+ Coverage   82.87%   83.84%   +0.96%
==========================================
  Files          52       52
  Lines        3008     3083      +75
  Branches       64       64
==========================================
+ Hits         2493     2585      +92
+ Misses        515      498      -17

Impacted Files
Coverage Δ

server/rest/tiles.py
95.34% <ø> (ø)
:arrow_up:

server/models/annotation.py
89.5% <100%> (+7.11%)
:arrow_up:

server/rest/annotation.py
50% <100%> (+3.65%)
:arrow_up:

server/models/annotationelement.py
96.42% <97.64%> (+13.92%)
: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 5f9db2e...77bea4b. Read the comment docs.

Closes issue #170.

@manthey if i'm reading this right,  it looks a lot like vector tiling? Are we sure there aren't any open source projects that we could pull modules out of that could potentially deliver more functionality with less maintenance overhead?

I've looked, and the only vector tiling I've seen that uses Mongo as a back-end requires using the Mongo 2dsphere index and the associated $near operator.  Unfortunately, Mongo's 2dsphere index only works with latitudes and longitudes (coordinates must be in the range of ([-180, 180], [-90, 90])).  The older 2d index where one could have arbitrary coordinates is deprecated (as of Mongo 2.6).  This means that we have to either do wacky things with stored coordinates, switch databases, or roll our own tiling of the vector features.

Although this is working as is (and I think I addressed all of @jbeezley's comments), we should probably have a visual indicator when not all elements are visible (or even requery to have a lower overlap when this occurs).


Update julie-lerman.yml

:warning: Validation status: warnings

File
Status
Preview URL
Details

rd/julie-lerman.yml
:warning:Warning

Details

rd/julie-lerman.yml

[Warning] the value Profile is undefined for metadata pagetype

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


joinChannel with Info

I wanna join channel with info.
I got error in iOS
Exception '-[__NSDictionaryM length]: unrecognized selector sent to instance 0x280d66840' was thrown while invoking joinChannel on target Agora with params

my code
await RtcEngine.joinChannel(channel, undefined, undefined, {
      userName: 'testUser1',
    })

the laster param is a string, I will fix it later.

@LichKing-2234
Thanks a lot 🙏
Do I have to write like this??
await RtcEngine.joinChannel(channel, undefined, undefined, `{
      userName: 'testUser1',
    }`)

the last param is not using for userName, you can't get the value which you have set.

@LichKing-2234
How can I join Channel with user's custom infomation

with your business service. our use the joinChannelWithUserAccount method.

@LichKing-2234
I see 🤔

@LichKing-2234
I want to see example joinChannelWithUserAccount😰
I tried this, but listner did not work('userJoined')
await RtcEngine.joinChannelWithUserAccount(
      channel,
      'TESTACCOUNT',
      'TESTTOKEN',
    )

have you set the correct token? you don't need to set token if you don't enable token in the dashboard console.

@LichKing-2234
this?? I did not set App Certificate.
I cannot discover Generate Temp Token😰

Generate Temp Token shows when you activate/enable the App Certificate
You can't however go back once you enable  the App Certificate

@LichKing-2234
I enabled App Certificate but listner did not work😰
    RtcEngine.init(config);

RtcEngine.on('userJoined', (data) => {
      console.log('userJoin' + JSON.stringify(data));

    });
    RtcEngine.on('userOffline', (data) => {

    });

    RtcEngine.on('localUserRegistered', (data) => {
      console.log('local' + JSON.stringify(data));
    });
    RtcEngine.on('userInfoUpdated', (data) => {
      console.log('userInfoUpdated' + JSON.stringify(data));
    });
    RtcEngine.on('joinChannelSuccess', (data) => {

      //If Local user joins RTC channel
      RtcEngine.startPreview(); //Start RTC preview
    });

    await RtcEngine.joinChannelWithUserAccount(
      channel,
      'Account',
      'MY TOKEN',
    )

your token is 'MY TOKEN'?
Generate Temp Token shows when you activate/enable the App Certificate.
you can test by using the Generate Temp Token

@LichKing-2234
yes . I Generate Temp Token

so, 'MY TOKEN'  means the generate temp token?

@LichKing-2234
'MY TOKEN' means the generate temp token, not a random string

the alpha version may have some unknown bugs. I suggest you use the beta version, here is the demo

@LichKing-2234
I begin using beta version, but issue happen😰
#221

@keisan1231 I have published a new version, you can try again.

@LichKing-2234 Thanks a lot🤓

@LichKing-2234 I tried.
engine.addListener('UserJoined', ..... did not work.
Both RtcEngine & RtcChannel class have joinChannel method, then what's difference??
I tried both, but both 'UserJoined' listener did not work.
I want to see sample code of lastest🙏
Is this maybe not lastest???

UserJoined means remote users joined.

if you do not need multiple channels, you just use the RtcEngine .
https://docs.agora.io/en/Interactive Broadcast/multiple_channel_android?platform=Android

@LichKing-2234 I tried.
engine.addListener('UserJoined', ..... did not work.
Both RtcEngine & RtcChannel class have joinChannel method, then what's difference??
I tried both, but both 'UserJoined' listener did not work.
I want to see sample code of lastest🙏
Is this maybe not lastest???

this demo is lastest.

UserJoined means remote users joined.

yes, I tried by using two devices.  But did not work.

if you do not need multiple channels, you just use the RtcEngine .

I see . Thanks🙏

this demo is lastest.

I'll try again later

@LichKing-2234
I tried 3.0.1-beta.5 version.
joinchannel
I cannot joinchannel with user info as ever😰
I can use joinChannel but optionalUid is allowed only 0.
engine.joinChannel(
   '006ad77514........',
      this.state.channelName,
      'ABC',  //  user info
      0,   //   I tried "1", but　UserJoined listener didnt work
    );

getUserInfoByUid
And I tried getUserInfoByUid in engine.addListener('UserJoined', data => {...
But I cannot get user information🤔
engine.addListener('UserJoined', data => {
        const {peerIds} = self.state;
        if (peerIds.indexOf(data) === -1) {
          self.setState({peerIds: [...peerIds, data]});
        }
        console.log(data);
        engine
          .getUserInfoByUid(0)
          .then(r => console.log(`${JSON.stringify(r)}`));  // {"uid":0,"userAccount":""}
      });

joinChannelWithUserAccount
And engine.joinChannelWithUserAccount can callengine.addListener('JoinChannelSuccess'...
but cannot call engine.addListener('UserJoined'.....

the optionalInfo param of joinChannel is not the userInfo. you can get more info from the comment about joinChannel method.
getUserInfoByUid should be called after joinChannelWithUserAccount

@LichKing-2234
joinChannelWithUserAccount did not call engine.addListener('UserJoined'.....
I want to implement that users can join a channel with users' info (userName, point, etc...)
Please tell me how to😰

Try this guide https://docs.agora.io/en/faq/string you apparently need to call registerLocalUserAccount first

@marqroldan
Thanks a lot!
I can register userAccount , but still engine.addListener('UserJoined'... was not called😰
So I cannot do video calling
Do I mistake addListener???

This works for me, I am receiving the event
      RtcEngine.registerLocalUserAccount(
        appid,
        'testuser',
      ).then(() => {
        console.log( registerLocalUserAccount 'success!');
        RtcEngine?.joinChannelWithUserAccount(
          tempToken,
          channel,
          'testuser',
        );
      });

@marqroldan
what is your version??   I use 3.0.1-beta.5
I tried your code but addListener('UserJoined'... was not called
I got normal joining(joinChannel method) successsfully, but did not joinChannelWithUserAccount method 😰
joinChannelWithUserAccount calls addListener('JoinChannelSuccess'......
but does not call addListener('UserJoined'.....

UserJoined is not for the local user but for the remote

@marqroldan
yes, I tried two devices. but UserJoined was not called after user in another device joined

Interesting, I'm using the web demo to join the channel for testing
https://webdemo.agora.io/agora-web-showcase/examples/Agora-Web-Tutorial-1to1-Web/?_ga=2.210623063.33363857.1588381252-1127249543.1583836640

@marqroldan
in only react native bug?

@LichKing-2234
I hope fixed . Thank you🙏

I use engine.joinChannelWithUserAccount(null, channelName, userAccount) and addListener
engine.addListener('JoinChannelSuccess', async (channel, uid, elapsed) => {
            console.log('RtcEngine onJoinChannelSuccess', channel, uid, elapsed);
            console.log(await RtcEngine.instance().getUserInfoByUserAccount(this.state.uid.toString()));
            console.log(await RtcEngine.instance().getUserInfoByUid(uid));
            this.setState({joinSucceed: true});
});

engine.addListener('UserJoined', (uid, elapsed) => {
            console.log('RtcEngine onUserJoined', uid, elapsed);
});

@keisan1231 it works fine. I join 2 users.
1 {"uid":<PHONE_NUMBER>, "userAccount": "88"}
2 {"uid":<PHONE_NUMBER>, "userAccount": "94"}

@LichKing-2234
Really!?  Latest version??

@keisan1231 yes, latest version

@LichKing-2234
Please show me full code🙏
maybe problem of lifecycle

        this.engine = await RtcEngine.create('yourAppId');
        this.engine.addListener('JoinChannelSuccess', async (channel, uid, elapsed) => {
            console.log('RtcEngine onJoinChannelSuccess', channel, uid, elapsed);
            console.log(await RtcEngine.instance().getUserInfoByUserAccount(this.state.uid.toString()));
            console.log(await RtcEngine.instance().getUserInfoByUid(uid));
        });
        this.engine.addListener('LeaveChannel', (stats) => {
            console.log('RtcEngine onLeaveChannel', stats.users, stats);
        });
        this.engine.addListener('UserJoined', (uid, elapsed) => {
            console.log('RtcEngine onUserJoined', uid, elapsed);
        });
        this.engine.addListener('UserOffline', (uid, reason) => {
            console.log('RtcEngine onUserOffline', uid, reason);
        });
        await this.engine.enableVideo();
        await this.engine.joinChannelWithUserAccount('yourToken', 'yourChannelName', 'yourUserAccount');

@LichKing-2234
Thanks a lot ❗
I try later👍

@LichKing-2234
I tried your code but UserJoined listener wasn't called😰
I cannot understand😰

because the uid is the same

you should use different userAccount on different client

@LichKing-2234

because the uid is the same

it works fine！！！
Thanks a lot🙏

@LichKing-2234
Can  parameter of userAccount have only word a-z, A-Z, 0~9???

You should not set userAccount with json.
You can refer to the method doc in .ts file.

@LichKing-2234
I want to use Japanese words as userAccount.
I hope such a update🙏

Sorry about it, We just support English. You can use your userId and get UserAccount from your system.
The user account. The maximum length of this parameter is 255 bytes. Ensure that you set this parameter and do not set it as null. Supported character scopes are:
* All lowercase English letters: a to z.
* All uppercase English letters: A to Z.
* All numeric characters: 0 to 9.
* The space character.
* Punctuation characters and other symbols, including: "!", "#", "$", "%", "&", "(", ")", "+", "-", ":", ";", "<", "=", ".", ">", "?", "@", "[", "]", "^", "_", " {", "}", "|", "~", ",".

@LichKing-2234
I see

@LichKing-2234
I want to get user info in UserJoined Listener.
But TypeError: undefined is not an object (evaluating 'info.userAccount')
code
this.engine.addListener('UserJoined', async (uid, elapsed) => {
      console.log('RtcEngine onUserJoined', uid, elapsed, 'No ', a);
      const {peerIds} = this.state; //Get currrent peer IDs
      const info = await RtcEngine.instance().getUserInfoByUid(uid);
      console.log(info.userAccount);
      if (peerIds.indexOf(uid) === -1) {
        //If new user
        this.setState({peerIds: [...peerIds, uid]}); //add peer ID to state array
      }
    });

is remote user join by userAccount?

@LichKing-2234
yes.
Sometimes successfully, but most of times failed by this error

@LichKing-2234
I got it successfully by using setTimeout()
this.engine.addListener('UserJoined', async (uid, elapsed) => {
      console.log('RtcEngine onUserJoined', uid, elapsed, 'No ', a);
      const {peerIds} = this.state; //Get currrent peer IDs
      setTimeout(async () => {
        const info = await RtcEngine.instance()
          .getUserInfoByUid(uid)
          .catch(e => console.log(e.toString()));
        console.log(info.userAccount);
        if (peerIds.indexOf(uid) === -1) {
          this.setState({peerIds: [...peerIds, uid]}); //add peer ID to state array
        }
      }, 2000);
    });


Add az vmss extension-upgrade start API to CLI

Resource Provider
Compute Resource Provider
Description of Feature or Work Requested
To add az vmss extension-upgrade start command to CLI to invoke extension upgrades on VMSS resources. This should invoke the extensionRollingUpgrade API in CRP to upgrade all extensions to their latest version on a VMSS.
Sample command: az vmss extension-upgrade start --name "VMScaleSetName" --resource-group "ResourceGroupName" --subscription "SubscriptionId"
Minimum API Version Required
2017-12-01
Swagger Link
https://github.com/Azure/azure-rest-api-specs/blob/23a83bfbab7f956695e0e0082b24ae08c99bf80f/specification/compute/resource-manager/Microsoft.Compute/stable/2020-06-01/compute.json#L4912
Target Date
09/20/2020

hi @chjaress is this Ignite ask?

hi @yungezz if possible yes :)

Supported in #15238, repoen the issue if there are any questions here.


[cubic-glyf] drawPoints and all-offcurve

Currently Glyph.drawPoints with a cubic all-offcurve contour generates something like this:
            ("beginPath", (), {}),
            ("addPoint", ((103, 181), None, False, None), {}),
            ("addPoint", ((125, 158), None, False, None), {}),
            ("addPoint", ((125, 104), None, False, None), {}),
            ("addPoint", ((103, 82), None, False, None), {}),
            ("addPoint", ((48, 82), None, False, None), {}),
            ("addPoint", ((26, 104), None, False, None), {}),
            ("addPoint", ((26, 158), None, False, None), {}),
            ("addPoint", ((48, 181), None, False, None), {}),
            ("endPath", (), {}),

which nowhere says cubic!
How should we resolve this?!

Should we add a curve point with None coordinates?

Should we add a curve point with None coordinates?

hm yeah, that would match the treatment of oncurve-less quadratic closed contour in the segment pen protocol (pen.qCurveTo((0,1), (2,3), (4,5), ..., None)).
But it would also break existing code that expects to be able to unpack the first argument as a (x, y) tuple.
I guess it's acceptable given that the existence of cubic contours with implied on-curves is a novelty. The UFO GLIF spec still only mentions that "curve" type segment can only be preceded by max 2 offcurves (unlike qcurve which has no such limit).
I wonder if, for consistency, we should also emit qcurve point with a None coordinates when a glyf contour only comprises quadratic offcurves? Maybe not, given the much higher risk of breaking existing tools.
An alternative solution might be to define an new explicit point type that describes a cubic offcurve point as opposed to the currently undefined "offcurve" type (which is actually a None value, given that an offcurve has no segmentType of its own, but it is supposedly qualified by an oncurve that comes after it... but some contours may have no on-curves hence our problem!)

An alternative solution might be to define an new explicit point type that describes a cubic offcurve point

Not bad. Another idea would be to add a kw arg to beginPath() that specifies the curve type for oncurve-less contours.

add a kw arg to beginPath() that specifies the curve type for oncurve-less contours

actually I think I like this option better. We only need to disambigate the curve type when there are no oncurves in a contour. The default can stay quadratic for backward compatibility.
Only problem with this approach is that one can't unconditionally beginPath and emit each point before knowing in advance whether the contour in question contains any oncurves at all, as that will be known only after reading all the points. Maybe that's ok. endPath could also have kwargs... 🤔


[plock-17]: load global shortcuts via settings

You can now update settings, update shortcuts, hit "load settings" and they work.

fixes #17


Update request: blanket 0.6.0 → 0.7.0

Package name: blanket
Latest released version: 0.7.0

Current version on the unstable channel: 0.6.0
Current version on the stable/release channel: 0.6.0

[x] Checked the nixpkgs pull requests

Notify maintainers
@jsierles onny

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Closing per #308444


[Feature request] Exit search mode on ESC

+1

Not sure if you're already aware, @krokofant, but you can use the Enter/Return key to exit search mode.

Thanks @jcarter, I'm aware but I feel the ESC is suitable to enter the editors normal mode. It's a common convention is it not?

I have just the minute released a new version with a command to exit Jumpy Mode. You can map this to ESC, by adding the following to your keybindings.json:
    {
        "key": "Escape",
        "command": "extension.jumpy-exit",
        "when": "editorTextFocus"
    }

This is also now documented in the README.

Golden! I really appreciate your work.

An update to this issue. The provided keybinding is blocking so it's blocking stuff intended for some other function as well.
An example is if you have an auto-complete box visible and want to close it. With the above mentioned keybinding it blocks the ESC signal to do what it should.
The when parameter should be specified more but I don't know how yet.

@wmaurer
To not make it block for me I used this:
    {
        "key": "Escape",
        "command": "extension.jumpy-exit",
        "when": "editorTextFocus && !suggestWidgetVisible"
    }

I don't know if some cases are still blocked.

Yes, it works perfectly!


Added Static middleware and supporting classes.

There are a lot of additions in here.
Some points:

One thing you'll want to look at is the BodyStream class.  It's used by the FileBody and TextBody classes.  BodyStream inherits StateMachine, which has some nice support for transition mapping and command eavesdropping.  This helped me figure out the differences between Kayak and ASP.NET host handlers recently.  It is, however, a re-implementation of some convenience methods on the Body class.  I'm not sure it's a complexity you want to take on, but I think it's worth it.

Static uses FileServer to handle the bulk of the file handling.

FileServer is written with future Sendfile middleware in mind, though this might take a different form when it finally materializes.

File contents are currently streamed in 64KB buffers.  I looked at some prior art for this.  Rack sends 8KB chunks.  Connect (via Node.js) sends 64KB chunks.  I went high.

Gate.Middleware took a dependency on Gate.Helpers.

Static is built to take a directory as its root and a white-list of URL's.  In the absence of this configuration (Static.Create()), it defaults to the "public" directory and marks all directories and files underneath as white-listed.

Note: This is mostly a move over from NRack to Gate with the exception of the BodyStream, which was created to specifically to track state of the OWIN body delegate.
Curious to see some feedback.

@bvanderveen said: "I kinda like that StateMachine class. Could you factor BodyStream so that it has-a StateMachine rather than is-a StateMachine?"
The BodyStream class kinda is a StateMachine.  Can you elaborate on why?
@bvanderveen said: "Hm, I don't want Gate.Middleware to take that dependency. Could you move what's needed into Gate.Middleware?"
I hear what you're sayin'.  I don't like taking on package dependencies, either.  I put things like HttpFormatExtensions, RangeHeader, BodyStream, FileBody, and TextBody in Gate.Helpers, because they have uses scoped outside the purpose of Gate.Middleware.  You could potentially use them in an app without requiring the Middleware library.  The other option is to put them in Gate, but I don't think that's right, either.  Thoughts?
@bvanderveen: "Also, could you include tests for all this stuff?"
Yeah, for sure!  Honestly, this has been sitting on my box for a few days (under a Gate Sample App project).  With some controversial changes in here, I wanted to get it out sooner than later.  You can think of this more as a "request for comments" instead of a "ready for merge" PR.  I have to migrate some tests over from NRack and create some new ones.  Will do.

Through some discussion and personal reflection, I conceded to all your requests.  :)
More tests should be added for some of the supporting files, but I've added a good start.  I'll add more when I get a little time.

This is in pretty decent shape now if you guys (@bvanderveen, @loudej) are ready to evaluate for merging.

Kevin, this code looks good, but could you make a new branch with a cleaned-up history and open a pull for that? This one seems to have some weird history—some commits repeated 3 times, and there are a few differences between them. Perhaps branch from the latest master and rebase/cherry-pick your changes back onto it?

Sure.  Check out Pull Request #48.  It surprises me that Git would have a problem merging this.  I pulled in upstream changes for the last commit.  That's why you see some dupes.  It's showing a rebase with all the previous commits on the branch re-applied.
FYI:
I grabbed the list from
git diff --name-only master static_middleware

Then, from master, I did:
git checkout -b static_middleware_clean
git checkout static_middleware -- <space-separated file list from above command>

That way, it just grabs the files from HEAD on static_middleware instead of bringing over any wacky commits.  Pull Request #48 should be pretty clean.

Sorry - I'm coming late into the conversation! I added comments in the new pull request. We should close this one probably.


Modifications to EMOS related to using all realizations as the predictor

Addresses Related to #854
Description
This PR includes:

Modifications to support the specification of the maximum number of iterations at the CLI. This is to support the usage of the realization option for the predictor of mean argument, where 200 iterations might not be sufficient to produce a stable solution, particularly if there are a large number of ensemble members.
Make a correction to the initial guess when the realization option is chosen. In this case, the b coefficients that are solved for can essentially be interpreted as weights for each of the realizations, so therefore, it makes sense for each realization to start off with a weight of 1./no_of_realizations. This initial guess gives the minimisation a better chance to find a stable solution.
Correct the datetime used in the CLI tests added for the estimate-emos-coefficients CLI.
A further PR will add the CLI desired in #854.

Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

I've now increased the default number of iterations to 1000 and refined the choice of initial guess, so that hopefully the solution will minimise sooner. Most of the code changes in the most recent commit are related to refactoring of the unit tests to try to make them a bit clearer by hopefully providing a simpler example, so that the coefficients generated can be more easily understood. Also hopefully the usage of statsmodels can be more easily understood in the unit tests, so I've added some tests that will be skipped depending upon whether statsmodels is in the environment.