break: Remove Promise#backtrace, use #reason.backtrace instead.

@lgierth why do we need a backtrace when a promise is fulfilled?  Also, can't we just use the backtrace on the exception when a promise is rejected?
If fulfill is called without arguments, this pull request does add a backtrace to the default error, which will now be an instance rather than a class.

This backtrace argument isn't just for error handling, but is meant for being able to trace fulfillment/rejection back to where it originated. This was usually the case when e.g. running in EventMachine, and the backtrace of each rejection was just a useless EM.run -> EM.execute_callback -> promise.rb -> block of app code. The method names are made up but you can probably guess what I mean.
I think I only ever had a use case for it when handling errors/rejection, but I though meh, eventually I'll want to trace fulfillments back to their origin, too.
Your call :P It's just clutter and added complexity if you don't have a use for it.

but is meant for being able to trace fulfillment/rejection back to where it originated

This is not only the case when an error is rescued somewhere, but also if something calls promise.reject by hand.

This is not only the case when an error is rescued somewhere, but also if something calls promise.reject by hand.

Is that because the backtrace wasn't set on the exception being passed into promise.reject?  E.g. promise.reject(RuntimeError.new(message))
I could have reject set the backtrace with reason.set_backtrace(caller) if reason.is_a?(Exception) && reason.backtrace.nil? if that is the source of the problem.

I think I was using reject incorrectly in that project (httpkit) -- the way I used it, it didn't always mean "error", sometimes it signaled a simple legitimate failure. E.g. like 404 is a legitimate HTTP response.
It might be fine to assume that reject is always caused by an error.

There are different levels of modifications that I could do for convenience, which would basically mimic raise:

Do nothing
reason.set_backtrace(caller) if reason.is_a?(Exception) && reason.backtrace.nil?
reason = reason.new if reason.is_a?(Class) && reason < Exception
reason = RuntimeError.new(reason) if reason.is_a?(String)

These support the use cases:

rescue promise.reject($!)
promise.reject(CustomError.new(message))
promise.reject(CustomError)
promise.reject(message)

If these coercions aren't done in reject, then .sync will do them implicitly through raise reason.
Based on what I am seeing in httpkit, I am seeing the third level of coercion (e.g. closed.reject(Errno::ETIMEDOUT)).  So should I support that, instantiating exception classes and setting a backtrace.

Note that the backtrace is currently incorrect if an exception is caught a then callback, as seen in this example:
p1 = Promise.new
p2 = p1.then { |value| 2 }
p3 = p2.then { |value| raise 'error in then' }
p1.fulfill(1) rescue nil  # p3.backtrace will get the backtrace from here

Updated to add support for instantiating error classes and setting a backtrace on them, e.g. for code like promise.reject(CustomError)


Truncate segments on broken header

Addresses parts of https://github.com/prometheus/prometheus/issues/3283 so we at least get a clean restart.

Test was a flake. LGTM :+1:


Documentation update needed

Running ephemerol requires the existence of a /tmp directory or you will receive FileIO errors when attempting to upload either a rules file or zip file.  Windows, for example, does not have this directory by default, so it must be created.
Very cool tool.

Thanks!
This should now be resolved by 323be83d395bbfa8213c27e08ec28abfe079b153
Let us know if this doesn't resolve the issue.


Run Amphetamine as a menu bar application

I would like to have the option to run Amphetamine run as a menu-bar application, so that when I cmd + tab through my applications it is not an option I have to roll over, and also so that it is not always sitting in the dock.

This has been available since v3.0. Amphetamine Preferences > General > (uncheck) Show Icon in Dock.


shields added into README.md file

Fixes Issue  #76

Changes proposed

       The changes involve the addition of shields in the README.md file

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] My change requires changes to the documentation.
[X] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@Cahllagerfeld I feel the same I should probably remove open issues and open PRs because it is readily available in Github. Instead, adding something else that is more helpful for contributors would be a good idea 😄

@Cahllagerfeld I have removed some badges and added some. Please take a look at it.
Previous

Present


To run example PPO experiment on academy_empty_goal scenario

All previous steps were successed.
But:
To run example PPO experiment on academy_empty_goal scenario, run python3 -m gfootball.examples.run_ppo2 --level=academy_empty_goal_close
Q:
An error happened:
~/gfootball/football/gfootball/examples [had (master)]$ python3 -m run_ppo2.py--level=academy_empty_goal_close
Traceback (most recent call last):
File "/home/XXX/anaconda3/lib/python3.7/runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/XXX/anaconda3/lib/python3.7/runpy.py", line 109, in _get_module_details
import(pkg_name)
File "/home/XXX/gfootball/football/gfootball/examples/run_ppo2.py", line 23, in
from baselines import logger
ModuleNotFoundError: No module named 'baselines'
But I have already installed module 'baselines'.
Thank U!

Are you sure you have the right version (for Python3 etc) installed?
Can you run:
pip3 install git+https://github.com/openai/baselines.git@master

Are you sure you have the right version (for Python3 etc) installed?
Can you run:
pip3 install git+https://github.com/openai/baselines.git@master

Yes pip3 install git+https://github.com/openai/baselines.git@master is ok!
And got :
Installing collected packages: numpy, six, cloudpickle, future, pyglet, scipy, gym, tqdm, joblib, click, opencv-python, baselines
Running setup.py install for baselines ... done
Successfully installed baselines-0.1.6 click-7.0 cloudpickle-1.2.1 future-0.17.1 gym-0.10.9 joblib-0.13.2 numpy-1.16.4 opencv-python-<IP_ADDRESS> pyglet-1.3.2 scipy-1.3.1 six-1.12.0 tqdm-4.32.1
But if run (base) XX@XX-XXX:~/gfootball/football/gfootball/examples [XX (master)]$ python3 -m run_ppo2 --level=academy_empty_goal_close
got : File "/home/XXX/anaconda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/XXX/anaconda3/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/XXX/gfootball/football/gfootball/examples/run_ppo2.py", line 23, in
from baselines import logger
ModuleNotFoundError: No module named 'baselines'

Do you happen to have multiple Python3 versions installed? It could be the case that you install baselines for, say Python3.6 and then use Python3.7 for running PPO.

Please reopen if this is still an issue.


Protocol v2, init handshake, init headers, etc.

Efforts to make tchannel-java talking to other languages & Hyperbahn:

Init req/res handshake should finish before other req/res can start: tchannel v2 protocol
Init req/res should have the host_port set when listen. It should use "<IP_ADDRESS>:0" otherwise for ephemeral ports.
Connection pool and peer selection support, e.g., should be able to prefer Hyperbahn affinity connections.

r: @willsalz @truncs @anson627
I feel this is enough for the first step to unblock Hyperbahn client & talking to other languages. The are still many places to improve. E.g., the connection pool can be a bottleneck for multi-threading. It can be relieved by read-write locks. But all for next steps.

Also your builds are failing.

@ShanniLi Few questions

Why can't the peer affinity logic live in the the hyperbahn module ?
Have you tested this with a Hyperbahn cluster ?
I am doing some benchmarking without this diff and its going to affect my work, how can we avoid it by making minimal changes and also to unblock hyperbahn ?

I have update the code to use ConcurrentHashMap per suggestions from @anson627 and @truncs

@truncs

Peer affinity logic lives in Hyperbahn. However, the client also needs to honor that.
This is just the very first step to get tchannel-java to talk in tchannel-v2. Talking to Hyperbahn will be the next step (may be several steps later depends on what's the next problem I find).
I think it would be better to do benchmark based on the latest tchannel protocol. What tchannel-java talks right now isn't really a tchannel protocol other languages (node, python, and go) respect. Let's makes sure it works at the protocol level before diving in to the perf work. I can help rebase your branch after the critical features/fixes are in.

lgtm


amalgkit batch jobs

It seems you are interested in running amalgkit in parallel; Here is the script with which I downloaded/quantified 161 SRAs using amalgkit.
amalgkit.sh.txt

Thanks! That might come in handy.


Update aws-java-sdk-bundle to 1.12.347

Updates com.amazonaws:aws-java-sdk-bundle from 1.12.330 to 1.12.347.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #2418.


Making the generated module names shorter

Is your feature request related to a problem? Please describe.
Some of our code cannot be deployed with stacks because of exceeding the 64 character limitation on resource module naming. This happens because stacks add 'YYYY-mm-dd-HH-MM-ss-xxxxx' string to the name of the bicep deployment it originates, and as we are using '${deployment().name}-something' to name our modules all of this together exceeds 64 characters sometimes.
Describe the solution you'd like
By default, shorten the string from 'YYYY-mm-dd-HH-MM-ss-xxxx' to something like 'YYYYmmddHHMMss-xxxxx' already 'saves' 5 characters.
It would be more convenient to give control to the caller if to make generated module names unique, or if it ok not to do so.
Describe alternatives you've considered
Changing our bicep modules implementation.
Additional context
Add any other context or screenshots about the feature request here.

Hi, I have the same issues.
Wanted to switch to stack as deployment but we are using a lot of Bicep Modules. We have created a structure where we use deployement.name() to create the names of our modules. But indeed seems stacks is adding a lot of stuff to the deployment name like "YYYY-mm-dd-HH-MM-ss-xxxx". Would there be a possibility to skip these added values in the deployment name from the stack?
Correlation id: '3d81457f-0e9f-446a-afd2-6c0986013c76'

Hi @antsok can you please share the correlationID for this failure if any?

Hi. Unfortunately, the original one is now not available, so I've reproduced it anew. The id: b6b90254-2ba0-4432-9c08-97a66d3cfb9c

What is the context of the creation of this Module (Assuming you are referring to BICEP modules)?

az stack sub create --name ado-aca ...  --template-file main.bicep ...
main.bicep deployment gets a long name from az stack. Example: ado-aca-2023-09-30-23-07-26-240ba
main.bicep with subscription target scope is using a bicep module with resource group scope.
module acr 'acr.bicep' = {
  scope: rg
  name: '${deployment().name}-acr-verylongstringattheendtomakeitfail' // <-- here
  params: {
    location: location
    enableAutoscaling: true
    ghBranch: ghBranch
  }
}

acr.bicep deployment uses the current module's deployment name to construct the submodule name.
The provided deployment name 'ado-aca-2023-09-30-23-07-26-240ba-acr-verylongstringattheendtomakeitfail' has a length of '72' which exceeds the maximum length of '64'.

it would be nice to add an optional parameter --deployment-name and give the user the choice to decide to be in control of the deployment name or not

Without the fix on deployment stacks end, the only other solution is to change our own bicep modules. This means time and investment, and we would probably not do it, sticking to the previous ways of deployment.

we need to add something to the deployment name so we don't automatically overwrite the previous deployment. Is there an acceptable number of characters you all would be comfortable with?

'YYmmddHHMMss' as an added prefix before/after the deployment name would be ok, imo.

So 12 characters is good enough, and 24 characters is too much? Anyone think 12 characters is still too much? Trying to understand what we need to optimize for.

Winning 12 characters back would already help us.
However, having a possibility of providing the unique prefix ourselves (we are using ADO pipeline run id for this) would be the best option. A separate parameter, maybe, that when provided is used instead of the current timestamp.

To follow up on this, we had a discussion to shorten the generated module to the following format:
<stackName>YYmmddhh<XXXX>
where XXXX is a Base36 hash for "extra" uniqueness to avoid conflicts.

I think this will be a good improvement already.

I will create a separate issue to track the ask of a command line argument to supply your own custom suffix.

hi @alex-frankel, just wondering ... why do you prefer passing a custom suffix and not a full deployment name? thanks

Let's discuss it in #133. I don't personally have a strong opinion on how it should work, I just wanted to keep the asks separate.
That being said, it can't just be a static name. It needs to be dynamic for each stack deployment or we would overwrite the previous deployment.

This change has been rolled out to all public regions, with government clouds to follow shortly.  The character count ended up appending 14 characters (down from 24).

This works smoothly for us now. Many thanks @alex-frankel and the team!


Parameterize librosa compatibility test

Closes #1346
I've parameterized the test_create_fb test, as mentioned in #1346:

We'd like to use parameterized for the functional librosa compatibility test here. This will make it easier to know which option is failing.
cc comment

I don't think the CI checks librosa < 0.7.2, but I installed librosa==0.7.1 here locally[1] and it works fine. 19 tests are generated rather than 25, and they all pass.
[1] With this fix for numba, so numba==0.48 rather than the default numba==0.52

I don't think the CI checks librosa < 0.7.2, but I installed librosa==0.7.1 here locally[1] and it works fine. 19 tests are generated rather than 25, and they all pass.
[1] With this fix for numba, so numba==0.48 rather than the default numba==0.52

@jcaw Could you try and see what happens with the skipped tests whenlibrosa==0.7.1? The skip is supposed to be related to facebook's internal system. However, I was doing a similar test because I thought I needed to apply the same guard for the recent changes, but it does not fail. If it does not fail on librosa==0.7.1 we should remove the skip logic.

@mthrok just tested with librosa==0.7.1 - the tests fail with this error:
        if norm is not None and norm != 1 and norm != np.inf:
>           raise ParameterError('Unsupported norm: {}'.format(repr(norm)))
E           librosa.util.exceptions.ParameterError: Unsupported norm: 'slaney'

thanks for checking with 0.7.1! it looks like the tests indeed fails for librosa <= 0.7.1 as you found out, or we need more investigation before removing the skip logic, which we'll leave to a potential follow-up. in either case, the goal of the PR is achieved, so I'll go ahead and merge the pull request :) thanks again!

Thank you!


Readme example does not compile

Tested on the latest version of the libraries (Rust: 1.54.0, hdf5: 0.7.1, ndarray: 0.15). Here is the compile error for the example in the README.md file.
error[E0277]: the trait bound `ArrayBase<OwnedRepr<Pixel>, Dim<[usize; 2]>>: From<&ArrayBase<OwnedRepr<Pixel>, Dim<[usize; 2]>>>` is not satisfied
  --> src/main.rs:30:22
   |
30 |           pixels.write(&arr2(&[
   |  ______________________^
31 | |             [Pixel { xy: (1, 2), color: RED }, Pixel { xy: (3, 4), color: BLUE }],
32 | |             [Pixel { xy: (5, 6), color: GREEN }, Pixel { xy: (7, 8), color: RED }],
33 | |         ]))?;
   | |__________^ the trait `AsRef<[_]>` is not implemented for `ArrayBase<OwnedRepr<Pixel>, Dim<[usize; 2]>>`
   |
   = note: required because of the requirements on the impl of `From<&ArrayBase<OwnedRepr<Pixel>, Dim<[usize; 2]>>>` for `ndarray::ArrayBase<ndarray::ViewRepr<&_>, ndarray::dimension::dim::Dim<[usize; 1]>>`
   = note: required because of the requirements on the impl of `Into<ndarray::ArrayBase<ndarray::ViewRepr<&_>, ndarray::dimension::dim::Dim<[usize; 1]>>>` for `&ArrayBase<OwnedRepr<Pixel>, Dim<[usize; 2]>>`

error[E0308]: mismatched types
  --> src/main.rs:39:9
   |
39 |         assert_eq!(colors.read_1d::<Color>()?, arr1(&[RED, BLUE]));
   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `ndarray::ArrayBase`, found struct `ArrayBase`
   |
   = note: expected struct `ndarray::ArrayBase<_, ndarray::dimension::dim::Dim<[usize; 1]>>`
              found struct `ArrayBase<OwnedRepr<Color>, Dim<[usize; 1]>>`
   = note: perhaps two different versions of crate `ndarray` are being used?
   = note: this error originates in the macro `assert_eq` (in Nightly builds, run with -Z macro-backtrace for more info)

The problem seems to be that version 0.7.1 requires ndarray  >=0.13,<0.15, but no warning is issued when there is a version mismatch.

This should be resolved when we release version 0.8 and only support ndarray 0.15


Add in slack emoji support

I added in some slack emoji support instead of using the default webhook icon.

@antonbabenko Great! What can I update with regards to formatting?

Merged, fixed formatting in https://github.com/terraform-aws-modules/terraform-aws-notify-slack/commit/969628e63759a65f966b056957debdd2f8ba01f7 and released v1.5.0.
Thank you for your contribution.


613 prepare release of v230

Closes #613

@deemp Thank you for your contribution! We appreciate your effort, but there are some areas for improvement. Unfortunately, we couldn't award points this time due to the lack of code review and absence of reviewer comments. In line with our policy, these oversights result in significant deductions. However, we've ensured a minimum reward of 4 points to recognize your effort. Your current balance stands at +36. We encourage you to address these points in future contributions to maximize your rewards. Keep up the good work, and we look forward to your next submission!


Mouse move/resize suggestion++

Currently using the Meta + Mouse Left BTN (move window) and Meta + Right BTN (resize window) : Removes the window from the tiling layout and leaves the window floating.

Meta + Mouse Left BTN (move window): Would be preferable if window would go back to tiled when released from the move and snap-tile to the closest boarder or swap location with the tiled window you released the one you are moving.
Meta + Right BTN (resize window): Would be preferable for the window to stay in tiled layout and change the size of the tiled window and those around it accordingly.
Example of kwin script that "accomplishes" the resizing portion: https://store.kde.org/p/1112552
Add a Shortcut option to tile/un-tile all windows on current desktop.

Hold shift to place windows back in the layout. The new version of the script (v1.0) no longer hooks into frameGeometryChanged but instead into tileChanged, which is better overall for stability but there is no longer any chance this feature will be implemented. See https://zeroxoneafour.github.io/polonium/faq.html
Resizing tiled windows seems to work with moving the layout around, but I may have to do some debugging to get it consistent with the script. I'll look into this.
Will work on it.


Option --blacklist has no effect

The use of --blacklist option to exclude sites from the submission has no effect.
For example<EMAIL_ADDRESS>was submitted with option  --blacklist Omaha but it ended up to run at Omaha site.
Ref: https://landscape.fnal.gov/kibana/goto/048c6bdd41105b5f1677e936ed368bf1

So right now there is a clause in requriements= to deal with (DESIRED_)Sites, but not Blacklist_Sites... so we probably
need to change
requirements  =... {%if site is defined and site != '' %} && ((isUndefined(target.GLIDEIN_Site) == FALSE) && (stringListIMember(target.GLIDEIN_Site,my.DESIRED_Sites))){%endif%}...

to
requirements  =... {%if site is defined and site != '' %} && ((isUndefined(target.GLIDEIN_Site) == FALSE) && (stringListIMember(target.GLIDEIN_Site,my.DESIRED_Sites))){%endif%} &&
 {%if blacklist is defined and blacklist != '' %} && ((isUndefined(target.GLIDEIN_Site) == FALSE) && (! stringListIMember(target.GLIDEIN_Site,my.Blacklist_Sites))){%endif%}

because we're taking the one into account but not the other...

The use of --blacklist still has no effect. Similar to the original contributor, I submitted several jobs with --blacklist Omaha, but they were passed to Omaha.
Ref: https://landscape.fnal.gov/kibana/app/kibana#/dashboard/ba047b90-b8ca-11e7-989a-91951b87e80a?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-12h,mode:quick,to:now))&_a=(description:'View%20jobs%20exit%20code,%20where%20they%20ran,%20and%20logs',filters:!(('$state':(store:appState),meta:(alias:!n,disabled:!f,index:'fifebatch-history-',key:pool,negate:!f,params:(query:fifebatch,type:phrase),type:phrase,value:fifebatch),query:(match:(pool:(query:fifebatch,type:phrase)))),('$state':(store:appState),meta:(alias:!n,disabled:!f,index:'fifebatch-history-',key:MATCH_EXP_JOB_GLIDEIN_Site,negate:!f,params:(query:Omaha,type:phrase),type:phrase,value:Omaha),query:(match:(MATCH_EXP_JOB_GLIDEIN_Site:(query:Omaha,type:phrase))))),fullScreenMode:!f,options:(darkTheme:!f),panels:!((gridData:(h:15,i:'1',w:40,x:0,y:0),id:'2f40f420-b8ca-11e7-989a-91951b87e80a',panelIndex:'1',type:visualization,version:'6.8.23'),(gridData:(h:10,i:'2',w:24,x:24,y:15),id:'569cca30-b8ca-11e7-989a-91951b87e80a',panelIndex:'2',type:visualization,version:'6.8.23'),(gridData:(h:10,i:'3',w:24,x:0,y:15),id:'65759a00-b8ca-11e7-989a-91951b87e80a',panelIndex:'3',type:visualization,version:'6.8.23'),(embeddableConfig:(columns:!(JobsubJobId,Owner,ExitCode,ExitSignal,MATCH_GLIDEIN_Site,MachineAttrMachine0,stdout,stderr),sort:!('@timestamp',desc)),gridData:(h:30,i:'4',w:48,x:0,y:25),id:'7e94c3c0-b8cb-11e7-989a-91951b87e80a',panelIndex:'4',type:search,version:'6.8.23'),(gridData:(h:15,i:'5',w:8,x:40,y:0),id:AWZpvkXbLj3wKbt0N_Vp,panelIndex:'5',type:visualization,version:'6.8.23')),query:(language:lucene,query:'Owner:jminock'),timeRestore:!f,title:'Fifebatch%20History',viewMode:view)

Um... there has been no new release including this bugfix yet, so unless you're
checking the code out of github, yes, it still isn't working.
--
Marc W. Mengel
Computer Services Senior Developer
Data Management and Applications
Fermi National Accelerator Laboratory
630 840 8256 office
www.fnal.gov
https://email.fnal.gov/owa/redir.aspx?C=JkVsWjSyHEuWV2kFJWNPyC9OyMKf7dEIp_yxPk9yfhfadE3mdLfI6T8Ztf8BVaJjzoIe8oJ5H0M.&URL=http%3A%2F%2Fwww.fnal.gov
Connect with us!
Newsletterhttps://email.fnal.gov/owa/redir.aspx?C=-29AtpIe40SAvGF4aiNr86KvVjap7dEIuxIdg8Q5bYa9sT7QTFVhxhVnl8zwXkEuBSuAQLRkEfg.&URL=http%3A%2F%2Fwww.fnal.gov%2Fpub%2Ftoday | Facebookhttps://email.fnal.gov/owa/UrlBlockedError.aspx | Twitterhttps://email.fnal.gov/owa/redir.aspx?C=-29AtpIe40SAvGF4aiNr86KvVjap7dEIuxIdg8Q5bYa9sT7QTFVhxhVnl8zwXkEuBSuAQLRkEfg.&URL=https%3A%2F%2Ftwitter.com%2FFermilabToday

From: jminock @.>
Sent: Friday, May 12, 2023 1:33 PM
To: marcmengel/jobsub_lite @.>
Cc: Marc W Mengel @.>; State change @.>
Subject: Re: [marcmengel/jobsub_lite] Option --blacklist has no effect (Issue #401)
The use of --blacklist still has no effect. Similar to the original contributor, I submitted several jobs with --blacklist Omaha, but they were passed to Omaha.
Ref: @.***',desc)),gridData:(h:30,i:'4',w:48,x:0,y:25),id:'7e94c3c0-b8cb-11e7-989a-91951b87e80a',panelIndex:'4',type:search,version:'6.8.23'),(gridData:(h:15,i:'5',w:8,x:40,y:0),id:AWZpvkXbLj3wKbt0N_Vp,panelIndex:'5',type:visualization,version:'6.8.23')),query:(language:lucene,query:'Owner:jminock'),timeRestore:!f,title:'Fifebatch%20History',viewMode:view)
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_marcmengel_jobsub-5Flite_issues_401-23issuecomment-2D1546128870&d=DwMFaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=mA4qFs7Hfq-joWhWXnsXfA&m=TfFdWZ4tsKlUujxU7wrC1nmm22FiZmnkruGFHsumxqO6D3tu_LHmIyIbE2yMiR4y&s=yaM2YubdWXDm6qEjYbhX6kz8QDFgF8FFGvc_zeCzccg&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAB6GV3EQZOKZCV7XH7SVTLXFZ677ANCNFSM6AAAAAAX5GUSMY&d=DwMFaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=mA4qFs7Hfq-joWhWXnsXfA&m=TfFdWZ4tsKlUujxU7wrC1nmm22FiZmnkruGFHsumxqO6D3tu_LHmIyIbE2yMiR4y&s=7V8dHFvJocOdHvKr5SdeDGZoWIeAcSszvSqgWmXlYV4&e=.
You are receiving this because you modified the open/close state.Message ID: @.***>


native gem support: add 3.2, drop 2.6

Closes #364

OK, precompiled gems on Darwin with Ruby 3.2 are suffering from the same problem as Nokogiri, with the Ruby process having already mapped /usr/lib/sqlite3.dylib and so:
$ ruby -r sqlite3 -e 'puts SQLite3::SQLITE_VERSION, SQLite3::SQLITE_LOADED_VERSION'
3.40.1
3.39.5

Going to have to do the -load_hidden workaround here, too.


Added tags in announcements. Announcements are categorized by tags.

Issue Reference
This PR addresses the Issue : Fixes #
Summarize
Please, describe what the PR does, the changes you have made.

Also, mention the key points if any to look into while code reviews

This is already satisfied by #1022.


[BUG] Datatable dont render after build

After build with npm run build or yarn build the datatable render only the message "There is no data yet, try to refresh..."
print:

Expected Behavior
Show datatable and data
Current Behavior
Component rendering only the message "There is no data yet, try to refresh..."
Possible Solution
Steps to Reproduce
1.fork/download this sample https://codesandbox.io/s/body-cell-override-example-for-o2xpreact-datatable-12rof
2.install dependencies
3.build
4. run
Context (Environment)

Detailed Description

Possible Implementation

Any news on this?

Sorry for delay, you can update to 1.1.27 ! @tuliocll @rchvalbo


Revert "qt5, libsForQt5: 5.12 -> 5.14 on darwin"

Reverts NixOS/nixpkgs#168524
The original change came without any testing on the target platform and caused a regression.
Fixes: #185615

The benefit of 5.14 over 5.12 was just that it was "less" end-of-life, but I'm highly in favor of reverting if there's any breakages being detected.

@tanshihaj Does this fix your issue?

No, downgrading to 5.12 will not help me since qt support aarch64-darwin starting from 5.15. So maybe better to bump it to 5.15?

@tanshihaj So you are saying that qt5 was broken on aarch64-darwin all this time? I have not seen any issue filed for that...

That’s been my understanding as well — other broken aarch64 stuff has messed up my home install so I haven’t had a chance to test on my new M2 machine though.

Search reveals there is one issue https://github.com/NixOS/nixpkgs/issues/130240
and this one is relevant https://github.com/NixOS/nixpkgs/issues/147817
as the clang 5 toolchain is likely the main cause for incompatibility

@veprbl yes qt5 was broken on aarch64-darwin all this time, moreover it was explicitly disabled here https://github.com/NixOS/nixpkgs/blame/7908242f926b3df62b46e98a69e88a2a60265125/pkgs/development/libraries/qt-5/modules/qtbase.nix#L377 until I've fixed build for Qt 5.15 here: https://github.com/NixOS/nixpkgs/commit/940b020fe73f1882163060e90300bc46732f19f9


[Feature]: Support Currencies in wrath

Describe the feature
Support currencies in wrath like they are in retail
Describe the implementation
I got this working in my local copy by adding the "Show Currencies" dropdown to the frames\classic\context.lua taken from the retail version of that file (this almost certainly will break in classic era/sod). Then I added a bunch of shims in frames\currency.lua for the wrath versions of the functions (the big one taken from weakauras code):
local C_CurrencyInfo_GetCurrencyListInfo = C_CurrencyInfo.GetCurrencyListInfo;
if (not C_CurrencyInfo_GetCurrencyListInfo) then
  C_CurrencyInfo_GetCurrencyListInfo = function(index)
    local name, isHeader, isExpanded, isUnused, isWatched, _, icon, _, hasWeeklyLimit, _, _, itemID = GetCurrencyListInfo(index)
    local currentAmount, earnedThisWeek, weeklyMax, totalMax, isDiscovered, rarity
    if itemID then
      _, currentAmount, _, earnedThisWeek, weeklyMax, totalMax, isDiscovered, rarity = GetCurrencyInfo(itemID)
    end
    local currencyInfo = {
      name = name,
      description = "",
      isHeader = isHeader,
      isHeaderExpanded = isExpanded,
      isTypeUnused = isUnused,
      isShowInBackpack = isWatched,
      quantity = currentAmount,
      trackedQuantity = 0,
      iconFileID = icon,
      maxQuantity = totalMax,
      canEarnPerWeek = hasWeeklyLimit,
      quantityEarnedThisWeek = earnedThisWeek,
      isTradeable = false,
      quality = rarity,
      maxWeeklyQuantity = weeklyMax,
      totalEarned = 0,
      discovered = isDiscovered,
      useTotalEarnedForMaxQty = false,
    }
    return currencyInfo
  end
end

local C_CurrencyInfo_SetCurrencyBackpack = C_CurrencyInfo.SetCurrencyBackpack;
if (not C_CurrencyInfo_SetCurrencyBackpack) then
  C_CurrencyInfo_SetCurrencyBackpack = function(index, backpack)
    if backpack then
      return SetCurrencyBackpack(index, 1)
    else
      return SetCurrencyBackpack(index, 0)
    end
  end
end

local C_CurrencyInfo_GetCurrencyListSize = C_CurrencyInfo.GetCurrencyListSize;
if (not C_CurrencyInfo_GetCurrencyListSize) then
  C_CurrencyInfo_GetCurrencyListSize = GetCurrencyListSize;
end

Also I had to change the font from xxx12Font to GameFontNormalSmall in currency.lua
I suck at lua and don't really have the long-term maintenance of this project in mind so I don't want to submit a PR like this with its obvious deficiencies. I just wanted to see how hard it would be.

I'll evaluate how to do this longer term -- but Cata is right around the corner, not sure about it. We'll see!


[Fix #9954] Fix an infinite loop error for Layout/HashAlignment

Fixes #9954.
This PR fixes an infinite loop error for Layout/HashAlignment when EnforcedStyle: with_fixed_indentation is specified for Layout/ArgumentAlignment.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looks good!


lparchive2epub: init at 1.2.3

This PR adds a package for lparchive2epub, a tool for downloading and converting LP Archive threads to EPUB files.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Apologies for the long turnaround!
Fixed the formatting of the meta field, and changed python3.pkgs to python3Packages. I also added pytestCheckHook, as tests were added/fixed upstream.


Revise Q4 Roadmap

Eliminate PI references
Signed-off-by: Rose Sakach<EMAIL_ADDRESS>

@balhar-jakub ; please review  / approve.  The existing copy is inconsistent with references to "PI" vs. "Q", example 22PI4 vs. 22Q4.  I modified all references of "PI" to read "Q" instead.


Issue loading

[05-03-2017 17:27] Error downloading account name, exception details: System.ApplicationException: Unable to deserialize object
at POEApi.Model.POEModel.GetProperObjectFromTransport[T](Stream networkData)
at POEApi.Model.POEModel.GetAccountName()
[05-03-2017 17:27] System.Exception: Error downloading account name, details logged to DebugInfo.log. Please open a ticket at https://github.com/Stickymaddness/Procurement/issues and include your DebugInfo.log
at POEApi.Model.POEModel.GetAccountName()
at Procurement.ViewModel.LoginWindowViewModel.<>c__DisplayClass23_0.b__0()
at System.Threading.Tasks.Task.Execute()

include your DebugInfo.log


Nodesphere.

https://github.com/nodesphere/nodesphere
Nodesphere is an ambitious project designed to connect the data sources that we care most about into a single unified dashboard:

Our social networks
Our curated collections
Our sense-making information
Nodesphere is a collection of isomorphic visualization-style interfaces. Each interface has particular strengths, so the user can switch between interfaces depending upon the data set and the desired interaction style.

Semantic Crunching
Nodesphere uses semantic crunching technology, which reduces data inputs into their atomic semantic nodes. Regardless of the data source, the important knowledge is digested into an interconnected graph, with appropriate connections between tags, categories, peoples' names, etc.
Open Graph Stack
The primary interface will be through the visualization layer, which allows filtering, sensemaking and visual navigation of all data (technology: javascript visualization)

Each individual or organization will have their own instance of a graph server, to store relationships between their most used data (technology: graph database)
All data may optionally be reflected to permanent distributed storage layers (technology: IPFS)

Status
Nodesphere is in active development, focussing on:

Multiple visualization interfaces
Morphing between visualizations
Reading data from multiple backends, starting with IPFS

Cool! Is there a beta version yet?

Nope!  Working on it : )
Mostly we are focusing on Core Network now:
https://github.com/core-network/core-network
which uses Nodesphere under the hood...
Thanks for noticing!  We'll keep you posted on progress.

Cool. Let's close this until it is up.


Feature/junit5 preparations

extracted the junit4 code into separate module
all dependencies are using this right now inside this project
module testing is empty so far, but will contain junit5 code later
this merge request is based on the need to hold the old module name testing
testing-junit4 will be used for the orig testing code base
teting-core will be used for the generic code that could be used for junit4 and junit5
testing will be the "home" for the new junit5 code base

for integration test-7 : https://github.com/fnproject/cli/pull/349
With this we can use junit4 as default until we are switching to jUnit5 as default.

Compilation failure in the integration tests - I'll have a look.

it should be integration test 7, that is using cli

deactivated integration test 3 and 7 to break the cycle between cli and fdk


IE8 Custom Event cleanup on Node Removal

Right now if a node has a custom event bound to it in IE8 (the workaround for custom events in IE8 is to assign a propertychange event to the document) and the node is removed, the document event binding will not be removed. We could add some code to .remove to do this?
Or maybe iterate over the event callbacks and check to see the source elements still exist during a trigger? Trigger might work ok since we wouldn’t have to handle user triggered behavior.
I should note that this is the reason a bunch of unbinds were added to the tests in 7b51125327d805817ef532602d511387b9c60731

Fixed with https://github.com/filamentgroup/shoestring/pull/134


[alerta] token header prefix

Back in December I merged a pull request that changed the alerta Authorization prefix from Bearer to Key because of an issue experienced by @emirozer https://github.com/influxdata/kapacitor/issues/1076.
I was running a custom version of alerta at the time so I assumed that I had changed Bearer to Key myself. On further inspection I realized that alerta officialy started using the Bearer prefix a year ago. https://github.com/guardian/alerta/commit/46e52094013f0c19a664f3aeb7ebdf1bfcbf64f4 which means @emirozer was using an older version of alerta.
I have been running a custom version of kapacitor for myself that uses Bearer so I haven't had any issues, but going forward there is a chance that someone will run into this again. I am proposing that the prefix be configurable. The default value would be Bearer.
I will work on a pull request in the coming days that addresses this. I will be sure to alert @emirozer if the pull request gets accepted.

@allen13 Thanks for being proactive on this. I'll keep an eye out for the PR.

@emirozer Once this commit goes through if you are still on the older version of alerta you can add this to your config:
[alerta]
token-prefix = "Key"

Thanks getting this merged in so quickly @nathanielc!

Sorry, speaking about new and old alerta... I actually managed to get kapacitor+alerta running only after I used token-prefix ="Key". before that it was Alerta: Token is invalid error all the time.
And alerta version is 5.0.4 which is the latest.

I agree with @v-zhuravlev. Systems integrating with Alerta should use "Key <api-key>" in the Authorization header, not "Bearer". "Bearer" is reserved for user logins like the web UI and alerta command-line tool. I suggest you change the default back to "Key".

Whatever the latest default for alerta is should probably be the default here. This is more of an issue with the alerta project changing back and forth.

@allen13 Just to avoid confusion, the Alerta authorization header has never changed. It was always "Key" for API keys and "Bearer" for the web UI.

Any update of this or reason it was not merged yet ? Still Alerta's default prefix is Bearer which is wrong and without change of kapacitord's config Alerta integration is broken.

I was going to prepare a PR cause I fell into the same trap. Can we get :eyes: on https://github.com/influxdata/kapacitor/pull/1753 and merge it cause the current default in kapacitor.conf is confusing.

This is confusing, I'm a long time alerta and kapacitor user and I had to change my config to something like that to have it work: (doing that for all those lovely web search engine users)
kapacitor.conf
[alerta]
  enabled = true
  url = "http://alerta:8080/api"
  token = "myToken"
  token-prefix ="Key"
  environment = "Development"
  origin = "kapacitor"


Use isolated scope with two-way binding in ng-sortable

I suggest retrieving the ngSortable options using a two-way binding in a local scope, instead of using attrs and scope.$eval.
This would remove the need for the 'ugly if'.

Thx.


.does_verb() does not match capitalization of parameter

For example, Now {{ users.does_verb(‘Come’) }} in a .docx template outputs Now come/comes.
This is inconsistent with .as_noun() which does match case.
.did_verb() may also have this issue. Have not tested.

For now there's a manual workaround with .capitalize()

This is upstream in https://github.com/jhpyle/docassemble/ but I'll see if I can make a pull request to fix it

Closing since this isn't appropriate to fix in this repo. Best we can do is a workaround or get it merged upstream.


@algo-builder/runtime has a missing module

Describe the bug
@algo-builder/runtime installed from npm is missing a module "zod".
Expected Behavior
I expected to be able to include the module in my Node.js JavaScript project without incurring an error.
Current Behavior
Attempting to require @algo-builder/runtime throws the following error:
Error: Cannot find module 'zod'

Possible Solution
Possibly need to include the appropriate version of zod in package.json.
Steps to Reproduce

Create a new project npm init -y
Install @algo-builder/runtime npm install @algo-builder/runtime
Create index.js and add const runtime = require("@algo-builder/runtime")
Run it node index.js

You should see the following error:
> node index.js
internal/modules/cjs/loader.js:883
  throw err;
  ^

Error: Cannot find module 'zod'

I tried installing "zod" like this npm install zod, but that results in a different error when running node index.js:
> node index.js
internal/modules/cjs/loader.js:438
      throw e;
      ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/src/ZodError' is not defined by "exports" in ...\algo-builder-test\node_modules\zod\package.json

Environment

Windows
Node.js version: 14.16.0
@algo-builder/runtime version: 1.2.0

Thanks so much.


Impossible to do an "IN" prepared statement query?

hey guys, very simple, but how can we achieve something like this?
It looks like the (@P1) is not accepted, and i do not know how to make it spit out the query after it is generated.
//Would like to use a prepared statement like this
pub const QUERY_LIST_CMDB_CI_SERVERS: &str = r#"
    SELECT
    cmdb.sys_id,
    cmdb.name,
    cmdb.u_tech_owner,
    u.email as 'pri_owner_email',
    cmdb.dv_u_tech_owner,
    cmdb.u_active,
    cmdb.short_description,
    cmdb.dv_u_cbt_owner,
    cmdb.u_cbt_owner
    FROM  cmdb_ci cmdb
    LEFT JOIN sys_user u on u.sys_id=cmdb.u_tech_owner
    WHERE cmdb.name IN (@P1)
    OR cmdb.sys_id IN (@P1)
    AND cmdb.u_active = 'true'
    AND (cmdb.sys_class_name = 'cmdb_ci_voice_hardware'
    OR cmdb.sys_class_name LIKE '%server%')
    AND cmdb.sys_class_name NOT LIKE '%web_server%'
    AND cmdb.sys_class_name NOT IN (
    'cmdb_ci_app_server_websphere',
    'cmdb_ci_app_server_jboss',
    'cmdb_ci_app_server_tomcat',
    'cmdb_ci_appl_license_server',
    'cmdb_ci_app_server_weblogic'
    );
"#;

// Instead of doing a format like this:

pub fn query_list_cmdb_ci_servers(keys: Vec<String>) -> String {
    format!(
        r#"
    SELECT
    cmdb.sys_id,
    cmdb.name,
    cmdb.u_tech_owner,
    u.email as 'pri_owner_email',
    cmdb.dv_u_tech_owner,
    cmdb.u_active,
    cmdb.short_description,
    cmdb.dv_u_cbt_owner,
    cmdb.u_cbt_owner
    FROM  cmdb_ci cmdb
    LEFT JOIN sys_user u on u.sys_id=cmdb.u_tech_owner
    WHERE cmdb.name IN ({0})
    OR cmdb.sys_id IN ({0})
    AND cmdb.u_active = 'true'
    AND (cmdb.sys_class_name = 'cmdb_ci_voice_hardware'
    OR cmdb.sys_class_name LIKE '%server%')
    AND cmdb.sys_class_name NOT LIKE '%web_server%'
    AND cmdb.sys_class_name NOT IN (
    'cmdb_ci_app_server_websphere',
    'cmdb_ci_app_server_jboss',
    'cmdb_ci_app_server_tomcat',
    'cmdb_ci_appl_license_server',
    'cmdb_ci_app_server_weblogic'
    );
    "#,
        keys.join(",")
    )
}

Hey! So, SQL Server doesn't really have support for vectors like some other databases, such as PostgreSQL do with their array operations, which would make it possible to write your query like you do in the first example.
So, we have the statement and we have the parameters. Using an IN statement, you must have a different statement for different array sizes, e.g. an array of size 2 needs IN (@P1, @P2) and size of 3 goes obviously as IN (@P1, @P2, @P3).
This means a constant query will not work, but you need to evaluate a new statement per array size, and pass the parameters flattened to the slice.
What we do in Prisma is we have a query builder, that will convert a convenient data structure into a query string and parameters. You can read how it's done in here: https://github.com/prisma/quaint/tree/master/src/ast
I warn you though of using that crate, or at least you should pin to a certain version of the master branch. We had to fork some of our dependencies ages ago and we can't really guarantee API stability anymore at this point. But, if you don't need super complex structures, what you can do is a method that should somehow go like this:
fn my_in_statement_query(params: &[something]) -> String {
    format!("... WHERE IN ({}) ...", params.iter().enumerate(), map(|(i, _)| format!("P{}", i)).collect::<Vec<_>>().join(","))
}

Now this one is not very efficient, and depending on your payload, it might use way more RAM in the server than you'd expect. Also, SQL Server has a size limit how many parameters you can have per query.  I think only PostgreSQL solved this in an elegant way, and you can do something like:
SELECT * FROM tbl WHERE id = ANY ($1);

Here the parameter can be an array, and you can use the same statement for all sizes of the array! If you find a nice way of doing the same in SQL Server, and we miss a feature that would enable it, please tell me!

Thank you for the amazing comment!
Yes this is something I've been dealing with for some time now but most people tend to shy away from this question.
I use and love Postgres, so the array types are just amazing for me, but sadly, at work we only use MSSQL.
That said, do you feel it is worth it to do the mapping?  It may be inefficient, but I think it will at least prevent sql injection, which is the whole reason for this post honestly.

In any ways you should design your application so you would not need large IN queries. If you have to, you might get an error if you have too many parameters, but you can handle that and it will not be a problem.
I highly highly recommend parameterizing everything that goes to the database, always!
And, SQL Server is great in some ways compared to PostgreSQL. But arrays it cannot support, so design your APIs accordingly.

Another option is to use relational operators. You can have your data for your in clause stored in a Vec<String> in Rust and then use serde_json::to_string to serialize this to a string and bind as a parameter. On the T-SQL side, you can use openjson(@P1) where @P1 is your serialized IN data. You can then join your current table to this one to do the filtering.
A basic example (from Rust you would bind a parameter rather than declaring one, I've just done this so the example can be run entirely in SQL):
declare @P1 nvarchar(max) = '[1,2,3,4,5]';
select a.value
from openjson(@P1) a

this returns the following data:

value

1

2

3

4

5

For your specific example, you could use something similar to this:
let filters_vec = vec!["filter1".to_string(), ...];
let filters = serde_json::to_string(&filters_vec).unwrap();
client.query(" SELECT
    cmdb.sys_id,
    cmdb.name,
    cmdb.u_tech_owner,
    u.email as 'pri_owner_email',
    cmdb.dv_u_tech_owner,
    cmdb.u_active,
    cmdb.short_description,
    cmdb.dv_u_cbt_owner,
    cmdb.u_cbt_owner
    FROM  cmdb_ci cmdb
    INNER JOIN openjson(@P1) a on a.value = cmdb.name or a.value = cmdb.sys_id
    LEFT JOIN sys_user u on u.sys_id=cmdb.u_tech_owner
    WHERE cmdb.u_active = 'true'
    AND (cmdb.sys_class_name = 'cmdb_ci_voice_hardware'
    OR cmdb.sys_class_name LIKE '%server%')
    AND cmdb.sys_class_name NOT LIKE '%web_server%'
    AND cmdb.sys_class_name NOT IN (
    'cmdb_ci_app_server_websphere',
    'cmdb_ci_app_server_jboss',
    'cmdb_ci_app_server_tomcat',
    'cmdb_ci_appl_license_server',
    'cmdb_ci_app_server_weblogic'
    )",
  &[&filters],
  )

there are also some alternatives to OPENJSON for instance STRING_SPLIT which may be useful if you don't already depend on serde_json.

Very interesting!! Will try it out

Another option is to use relational operators. You can have your data for your in clause stored in a Vec<String> in Rust and then use serde_json::to_string to serialize this to a string and bind as a parameter. On the T-SQL side, you can use openjson(@P1) where @P1 is your serialized IN data. You can then join your current table to this one to do the filtering.
A basic example (from Rust you would bind a parameter rather than declaring one, I've just done this so the example can be run entirely in SQL):

declare @P1 nvarchar(max) = '[1,2,3,4,5]';

select a.value

from openjson(@P1) a

this returns the following data:
| value |
| --|
| 1 |
| 2 |
| 3 |
| 4 |
| 5 |
For your specific example, you could use something similar to this:

let filters_vec = vec!["filter1".to_string(), ...];

let filters = serde_json::to_string(&filters_vec).unwrap();

client.query(" SELECT

    cmdb.sys_id,

    cmdb.name,

    cmdb.u_tech_owner,

    u.email as 'pri_owner_email',

    cmdb.dv_u_tech_owner,

    cmdb.u_active,

    cmdb.short_description,

    cmdb.dv_u_cbt_owner,

    cmdb.u_cbt_owner

    FROM  cmdb_ci cmdb

    INNER JOIN openjson(@P1) a on a.value = cmdb.name or a.value = cmdb.sys_id

    LEFT JOIN sys_user u on u.sys_id=cmdb.u_tech_owner

    WHERE cmdb.u_active = 'true'

    AND (cmdb.sys_class_name = 'cmdb_ci_voice_hardware'

    OR cmdb.sys_class_name LIKE '%server%')

    AND cmdb.sys_class_name NOT LIKE '%web_server%'

    AND cmdb.sys_class_name NOT IN (

    'cmdb_ci_app_server_websphere',

    'cmdb_ci_app_server_jboss',

    'cmdb_ci_app_server_tomcat',

    'cmdb_ci_appl_license_server',

    'cmdb_ci_app_server_weblogic'

    )",

  &[&filters],

  )

there are also some alternatives to OPENJSON for instance STRING_SPLIT which may be useful if you don't already depend on serde_json.


Typo "Azure files"→"Azure Files"

https://learn.microsoft.com/en-us/azure/architecture/example-scenario/hybrid/azure-files-on-premises-authentication https://github.com/microsoftdocs/architecture-center/blob/main/docs/example-scenario/hybrid/azure-files-on-premises-authentication.yml #PingMSFTDocs

Docs Build status updates of commit d412d03:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/hybrid/azure-files-on-premises-authentication.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@EdPrice-MSFT
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#Sign-Off


Added warning for 3D models with improper scaling

Adds a warning for models like in #57895 from Mixamo when a skeleton's parent's scale is less than 1 in any axis.

I've only seen this issue in models from Mixamo but according to this https://github.com/godotengine/godot/issues/57895#issuecomment-1072943105 checking if the parent's scale is less than 1 might not be enough.

Instead of checking if scale < 1.0. You can try basis.get_scale() != Basis().get_scale(). I'm worried this may spam.

Instead of checking if scale < 1.0. You can try basis.get_scale() != Basis().get_scale(). I'm worried this may spam.

Done.

Let me get back to you. I think the error happens when the mesh's Ref's binds are invalid like 0.01. I want to preserve the non uniform scaling usecase.

Please reconsider the wording of the warning message. Whilst Blender is a wellknown software, it isn't the only software capable of rescaling models. "Program like Blender" doesn't properly encapsulate what the program needs to be capable of to perform the action. The user shouldn't be prompted to find software like Blender but a tool that can resize models i.e. 3d modelling software.
Also, do we have any similar warnings? Any prior guidelines on how these should be worded and what kind of tense/tone should be used? (If not maybe such should be decided upon) I can only recall the warning tooltip for when a collision object has no shape:
This node has no shape, so it can't collide or interact with other objects. Consider adding a CollisionShape or CollsionPolygon as a child to define its shape.
At least in this instance, it is assumed a shape is desired to avoid potential issues, but the wording is only suggestive in nature. Maybe the warning should also be worded as such. For example "Consider setting the scale to (1,1,1) in an X to avoid issues." or "Consider importing a model with scale (1,1,1) to avoid issues." (Better suggestions are welcome).

Consider authoring the scene with scale (1,1,1) to avoid issues. Maybe you can improve on this wording.

Change text to Consider authoring the scene with scale (1, 1, 1) to avoid issues.

Can you look into verifying that the scale of the skins is not bigger than 2? 10? instead of checking the skeleton's parent.
Here's the screenshot of the case.

Standard operation to fix Character.gltf:

Open in creator tool of choice.
Clear scale on the scene
Note the character is 100 times bigger
Import as 0.01 scale on scene
Check that each Mesh's scale is around 1 in the skin bind.

I'll do this as soon as possible, I'm a bit busy with college work now.

While authoring scenes at scale (1,1,1) is certainly nice to do, I don't think the engine should fail on other scene sizes.
Instead of adding a warning hinting at "issues", we should file bugs about issues that come up including reproduction cases and discuss solutions. (For example, I recall there was a problem LOD system caused by scale in the bind-pose-matrix, due to FBX conversion. I don't know if there is an open issue about this one)
Regarding import warnings, unless they are meant only for development purposes, they probably should be displayed somewhere visible to the user, like the import dialog.

I've tried to do fix the skins in GDScript first but this is what happens now:
https://user-images.githubusercontent.com/18116695/160252972-985e8de6-ab2e-478e-b4e5-0dcfe15f1c8d.mp4
func fix_skins():
	var character_skin_binds: Array[Dictionary] = []
	var hair_skin_binds: Array[Dictionary] = []
	var character_skin_bind_count := character_mesh.skin.get_bind_count()
	var hair_skin_bind_count := hair_mesh.skin.get_bind_count()

	for i in range(character_skin_bind_count):
		var bind_pose := character_mesh.skin.get_bind_pose(i)
		bind_pose = bind_pose.scaled(bind_pose.basis.get_scale().inverse())
		var bind_name := str(character_mesh.skin.get_bind_name(i))
		character_skin_binds.push_back({"name": bind_name, "pose": bind_pose})
	for i in range(hair_skin_bind_count):
		var bind_pose := hair_mesh.skin.get_bind_pose(i)
		bind_pose = bind_pose.scaled(bind_pose.basis.get_scale().inverse())
		var bind_name = str(hair_mesh.skin.get_bind_name(i))
		hair_skin_binds.push_back({"name": bind_name, "pose": bind_pose})

	character_mesh.skin.clear_binds()
	hair_mesh.skin.clear_binds()

	for i in range(character_skin_bind_count):
		var bind_pose: Transform3D = character_skin_binds[i]["pose"]
		var bind_name: String = character_skin_binds[i]["name"]
		character_mesh.skin.add_named_bind(bind_name, bind_pose)
	for i in range(hair_skin_bind_count):
		var bind_pose: Transform3D = hair_skin_binds[i]["pose"]
		var bind_name: String = hair_skin_binds[i]["name"]
		hair_mesh.skin.add_named_bind(bind_name, bind_pose)

I don't have any experience with skinned meshes, so should I continue with this?

I think the right approach is to simply apply the skeleton rest to the mesh before computing the LOD. This should ensure it works in all cases.

I think the right approach is to simply apply the skeleton rest to the mesh before computing the LOD. This should ensure it works in all cases.

As I understand it, the problem is that some skin binds have a scale of around 100 for this glTF file. I've tried setting the skin using Skeleton3D::create_skin_from_rest_transforms() but it just returns the same oversized skin.

Closing this as it's better to fix the issue instead of just warning.

The LOD issue is one thing. People's DCC software outputting bad meshes will still continue to happen, so a warning would still be useful.


Support IAM conditions in Cloud Bigtable.

Once we merge #2722 (or something similar) we will get the new IAM protos:
At that time we should change the IAM classes as outlined in #1833. This is the tracking bug for Cloud Bigtable of that work. We need to:

[ ] Change bigtable::InstanceAdmin::GetIamPolicy() and bigtable::InstanceAdmin::SetIamPolicy() to return an error if the protos received from the wire are not representable using a google::cloud::IamPolicy.

[ ] Create a new class in google::cloud::bigtable (for now) to represent google.iam.v1.Binding protos, this class should be a faithful representation of the proto, including the new fields.

[ ] Create a new class in google::cloud::bigtable (for now) to represent google.iam.v1.Policy protos. This class should be a faithful representation of the proto, no attempts to reindex the data in any way. Helper functions to add, remove, and change bindings are acceptable, but there should be no "index by role" or "index by role, condition, etc."

[ ] Create new functions in google::cloud::bigtable::InstanceAdmin to return the new classes represented above. Naming is important:

InstanceAdmin::SetIamPolicy can simply be an overload with the new types.
InstanceAdmin::SetIamPolicyUninterpreted is not a great name, but start exploring from there.

The first step in this bug is to create child issues for each step. We should expect the usual unit, integration, and examples for these new functions.

FYI, this TODO still exists in the code:
https://github.com/googleapis/google-cloud-cpp/blob/e3f5ae369372510838a7dbeb4bf18742b8952911/google/cloud/storage/internal/bucket_requests.cc#L260


Turrican PAL

It seems there some issues with the scaling on this game..

Intro legs of turrican are cut off
Ingame small part of thr bottom bit of the gfx are cut out

It was off by about 3px, fixed. Note that we consider it an acceptable tradeoff that title screens etc that don’t affect gameplay will sometimes lose a few pixels, see the explanation and rationale here: https://amiga.vision/5x
(If we were not OK with this, a large number of games would not be able to run at 5× scale under 1080p/4K)


tool should validate that discovered java version is 1.8 before running certain SDK tools

Steps to Reproduce

clone this repo
install SDK command-line tools by unzipping the file available at developer.android (for Linux; I'm on Ubuntu 18.04) to $HOME/cmdline-tools. This provides me with an sdkmanager executable at $HOME/cmdline-tools/tools/bin/sdkmanager
prep SDK by mimicking the commands listed in the file dev/bots/download_android_sdk.sh in this repo:

yes | $ANDROID_SDK_ROOT/cmdline-tools/tools/bin/sdkmanager --licenses
$ANDROID_SDK_ROOT/cmdline-tools/tools/bin/sdkmanager tools
$ANDROID_SDK_ROOT/cmdline-tools/tools/bin/sdkmanager platform-tools
$ANDROID_SDK_ROOT/cmdline-tools/tools/bin/sdkmanager  "platforms;android-28" \
    "build-tools;28.0.3" \
    "platforms;android-27" \
    "build-tools;27.0.3" \
    "extras;google;m2repository" \
    "extras;android;m2repository"

where for me ANDROID_SDK_ROOT is $HOME/Android. As far as I can tell, all goes well with these.

run <path-to-flutter> doctor -v; this finds SDK all right, but cries about licenses (see the output below). I am instructed to run doctor --android-licenses.

Finally, this last command produces an error:

$ export ANDROID_HOME=~/Android && ~/repos/flutter/bin/flutter doctor --android-licenses
---
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
        at com.android.repository.api.SchemaModule$SchemaModuleVersion.<init>(SchemaModule.java:156)
        at com.android.repository.api.SchemaModule.<init>(SchemaModule.java:75)
        at com.android.sdklib.repository.AndroidSdkHandler.<clinit>(AndroidSdkHandler.java:81)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:73)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema
        at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
        at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
        ... 5 more

Logs

Output to <path-to-flutter> doctor -v:
[✓] Flutter (Channel master, 1.21.0-6.0.pre.180, on Linux, locale en_US.UTF-8)
    • Flutter version 1.21.0-6.0.pre.180 at $HOME/repos/flutter
    • Framework revision 428be90768 (3 hours ago), 2020-08-04 14:31:05 -0700
    • Engine revision 033dd45607
    • Dart version 2.10.0 (build 2.10.0-2.0.dev 0f0e04ec3a)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at $HOME/Android
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = $HOME/Android
    • ANDROID_SDK_ROOT = $HOME/Android
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-post-Ubuntu-0ubuntu118.04.1)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[!] Connected device
    ! No devices available

! Doctor found issues in 3 categories.

If the Android SDK tools require classes from Java 1.8. Normally this is provided by a JRE downloaded alongside Android Studio, but if you are setting up the SDK yourself, you will need to make sure that JAVA_HOME points to a 1.8 SDK path while running android licenses

Thank you! Both of these approaches worked on this Ubuntu 18.0.4 machine:

uninstalling openjdk-11-jre and installing openjdk-8-jre, and
keeping both but exporting JAVA_HOME to /usr/lib/jvm/java-1.8.0-openjdk-amd64 before running flutter doctor --android-licenses

The doctor output looks fine now:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 1.21.0-6.0.pre.180, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[!] Android Studio (not installed)
[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

I do have to run it in a terminal where I've exported JAVA_HOME to 1.8 though: the issues resurface with openjdk-11.

FWIW this is generally an area of the tool I'm working on improving. We shouldn't need to require users uninstall newer Java versions.

Another potential fix is surfacing an env variable just for android tools, falling back to JAVA_HOME as a last resort

Hi, first-time Flutter user here. I want to put in a huge vote for this issue: flutter doctor should be updated to check for Java == 1.8. It should inform the user if an incorrect version was found, and should give some suggestions about what to do.
My experience today (a bad first experience):

echo $JAVA_HOME --> /Library/Java/JavaVirtualMachines/jdk-15.jdk/Contents/Home
flutter doctor --> Android license status unknown. Run flutter doctor --android-licenses to accept the SDK licenses.
flutter doctor --android-licenses --> Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
(many hours of searching)
/usr/libexec/java_home -v '1.8*' --> /Library/Java/JavaVirtualMachines/jdk1.8.0_131.jdk/Contents/Home
export JAVA_HOME='/Library/Java/JavaVirtualMachines/jdk1.8.0_131.jdk/Contents/Home'
flutter doctor --android-licenses--> works as expected

My experience is very common. Here is an incomplete list of issues filed by users who encountered this same challenge:
https://github.com/flutter/flutter/issues/8438#issuecomment-356440563
#16025
https://github.com/flutter/flutter/issues/16065#issuecomment-391563094
#16280
#16452
https://github.com/flutter/flutter/issues/23732#issuecomment-434267450
https://github.com/flutter/flutter/issues/25753#issuecomment-450568874
#28230
#28504
#31116
https://github.com/flutter/flutter/issues/39823#issuecomment-538989010
#47118
#51155
#51670
https://github.com/flutter/flutter/issues/51712#issuecomment-599653138
#56572
#56778
#56798
#57017
#57372
#59857
#61354
#65414
#66458
#66907
#67050
#67128
https://stackoverflow.com/questions/46402772/failed-to-install-android-sdk-java-lang-noclassdeffounderror-javax-xml-bind-a/50285218

(In the past, flutter doctor was altered to check for Java >= 1.8. See issue #5880 and PR #41989. Hopefully the same infrastructure can be re-used to check for Java == 1.8.x.)

Thank you! Both of these approaches worked on this Ubuntu 18.0.4 machine:

uninstalling openjdk-11-jre and installing openjdk-8-jre, and
keeping both but exporting JAVA_HOME to /usr/lib/jvm/java-1.8.0-openjdk-amd64 before running flutter doctor --android-licenses

The doctor output looks fine now:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 1.21.0-6.0.pre.180, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[!] Android Studio (not installed)
[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

I do have to run it in a terminal where I've exported JAVA_HOME to 1.8 though: the issues resurface with openjdk-11.

Yes, I can fix it via change the JAVA_HOME environment.
C:\Users\Liu.D.H>flutter doctor --android-licenses
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
        at com.android.repository.api.SchemaModule$SchemaModuleVersion.<init>(SchemaModule.java:156)
        at com.android.repository.api.SchemaModule.<init>(SchemaModule.java:75)
        at com.android.sdklib.repository.AndroidSdkHandler.<clinit>(AndroidSdkHandler.java:81)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:73)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema
        at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
        at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
        ... 5 more

C:\Users\Liu.D.H>java -version
openjdk version "11.0.8" 2020-07-14
OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.8+10)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.8+10, mixed mode)

C:\Users\Liu.D.H>where java
C:\Program Files\AdoptOpenJDK\jdk-<IP_ADDRESS>-hotspot\bin\java.exe
C:\Program Files\Java\jdk1.8.0_191\bin\java.exe

C:\Users\Liu.D.H>set PATH=C:\Program Files\Java\jdk1.8.0_191\bin\;%PATH%

C:\Users\Liu.D.H>where java
C:\Program Files\Java\jdk1.8.0_191\bin\java.exe
C:\Program Files\AdoptOpenJDK\jdk-<IP_ADDRESS>-hotspot\bin\java.exe

C:\Users\Liu.D.H>flutter doctor --android-licenses
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
        at com.android.repository.api.SchemaModule$SchemaModuleVersion.<init>(SchemaModule.java:156)
        at com.android.repository.api.SchemaModule.<init>(SchemaModule.java:75)
        at com.android.sdklib.repository.AndroidSdkHandler.<clinit>(AndroidSdkHandler.java:81)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:73)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema
        at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
        at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
        ... 5 more

C:\Users\Liu.D.H>java -version
java version "1.8.0_191"
Java(TM) SE Runtime Environment (build 1.8.0_191-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.191-b12, mixed mode)

C:\Users\Liu.D.H>echo %JAVA_HOME%
C:\Program Files\AdoptOpenJDK\jdk-<IP_ADDRESS>-hotspot\

C:\Users\Liu.D.H>set JAVA_HOME=C:\Program Files\Java\jdk1.8.0_191\

C:\Users\Liu.D.H>flutter doctor --android-licenses
All SDK package licenses accepted.======] 100% Computing updates...

C:\Users\Liu.D.H>

Until the Android SDK supports java > 1.8, flutter doctor needs to handle checking this. It's a guaranteed chokepoint for both new developers and even experienced ones setting up a new machine. The expectation is that flutter doctor will tell you if anything is out of order

Just had this error, solved by downloading the Android SDK Command-line Tools (latest) on Android Studio, under Preferences > Appearance & Behavior > System Settings > Android SDK > SDK Tools and re-running flutter doctor --android-licenses

IMO downgrading jdk is a bad plan

This worked for me!
https://github.com/flutter/flutter/issues/56778#issuecomment-834062619


resolve references before validating the discriminator

fixes #1554

What issue does this pull request resolve?
What changes did you make?
Is there anything that requires more attention while reviewing?

Great - added some comments, it’s quite a common request.
please also update the docs to mention that single $refs (that is, without siblings) are now supported as well.

CI runs a bit more combinations of options than run locally by default, it uses:
AJV_FULL_TEST=true npm test
Maybe that’s the difference?
See the test output in CI.

So you can try locally with the same env variable

Thanks for the change, it definitely improves the usability of discriminator - only supporting inlined schema was an annoying restriction...
There couple of comments I wanted to amend - let me know if it's ok I do it?

Sure, feel free.


add anthropic support

Builds on top of
https://github.com/holoviz/lumen/pull/697
Closes https://github.com/holoviz/lumen/issues/604

Nice!


chore(ui): Preview URL for design team to review PRs

Preview URL for design team to review PRs
Type: Chore
Description: The design team would like to preview changes to the UI associated with PRs before they are deployed. Spike into an implement (Netlify?)
User Story:
As a designer, I want to preview changes to the UI before they are deployed so that we can make corrections before deploying to staging.
Acceptance Criteria:

[ ] Deploy UI changes easily at a URL accessible by design team

We will use Leapfrog-04 as a "development" environment that we can deploy PRs to for product to check. Leapfrog-demo will be the "stable" demo environment with latest release deployed.


Brew installation failed

Mac OS 10.12.6
brew update
brew cask install eqmac
==> Satisfying dependencies
==> Downloading https://github.com/romankisil/eqMac2/releases/download/v2.0.7/eqMac2.dmg

curl: (22) The requested URL returned error: 404 Not Found
Error: Download failed on Cask 'eqmac' with message: Download failed: https://github.com/romankisil/eqMac2/releases/download/v2.0.7/eqMac2.dmg

Why does Brew pick up unreleased builds? Should only allow 2.0.6 to be downloaded. Will look into this

Should be fixed by PR https://github.com/caskroom/homebrew-cask/pull/40854#pullrequestreview-77059835

Tested. Installation works now. Thanks for reporting :) Closing

Hey everyone. Excuse me for the lack of updates. I have been really busy on the new version of eqMac. I am happy to announce an open beta release.
You can download it here: https://eqmac.app
There are a lot of known bugs that I could not fix just yet:

Driver not installing from the first try. Sometimes need to restart eqMac a couple times.
Complex Device switching logic is not perfect (for example when switching off the device that's currently playing, plugging in new device etc.)
UI takes long to load (a lot can be done to improve that)
UI dimensions are not synching properly
There's a noticeable audio delay in the playback, so not suitable for video yet (I know where the problem lies just haven't had the time to fix yet)
Sometimes the playback doesn't start, in that case please try to switch the output to the device you want and eqMac should try and build the audio pipeline again.

So as you can see there's still some work to be done but at least it's a completely modern tech stack that I am fully willing to work on: 100% Swift, User Space driver and a Web UI, instead of C++, Kernel Drivers. As I said, should work on Catalina and should not have any pesky ear drum blasting volume bugs (at least I didn't come across any).
I'm open to feedback and issue reports here: https://github.com/bitgapp/eqmac
That repo will become the main repo for eqMac and this repo will be archived eventually.
And yes it's called just eqMac, not eqMac3 :) And it will stay that way forever.

Hey everyone 👋
I will be open sourcing the new version of eqMac, therefore, I am closing this issue and repository as I will be transfering nodeful/eqMac2 repo to bitgapp/eqMac
Please bear with me for the next couple of days as I setup that new repo. The eqMac2 code will be living in the old-eqMac2 branch if anyone is interested in referencing it for whatever reason.
Please try out the new version and report bugs to the new repo.
Thanks for using eqMac.


Move code to Plugins folder

Maybe it's time to do this?

Is there any reason?

Do not litter the main folder


WebSession: support for multiple emails

Originally on 2013-03-21
Up to know the email address played a central role in Invenio, often considered as unique identifier for a user.
With Invenio supporting multiple authentication method (possibly concurrently), it is very likely that user have multiple email addresses that he might desire to be recognized in Invenio.
This require amending the current database tables structure in particular by introducing a new table userEMAIL
e.g.:
CREATE TABLE IF NOT EXISTS userEMAIL (
  id_user int(15) unsigned NOT NULL,
  email varchar(255) NOT NULL default '',
  priority int unsigned NOT NULL,
  PRIMARY KEY email (email),
  KEY id_user (id_user),
) ENGINE=MyISAM;

The email column in user table could be kept in order to maintain track of the preferred email address for the user for communications.
Special care must be devoted in adapting APIs (e.g. collect_user_info and friends in order to take into consideration the availability of more than one email address.)

There is no PR for this issue, hence closing it as per the legacy code base freeze; it is addressed in master code base differently.


Accessing the defect dojo in the Vms

hi team
I have installed the defect dojo using the virtual machine in the gcp using the docker.But now I want to access the defect dojo dashboard to integrate some of the tool reports in order to track them efficiently
could you please help me with that

Hi @Nsai1997, could you please ask in the OWASP Slack channel this question or use the official support. This is not an issue.
@mtesauro I guess this can be closed.


ReferenceError: deepClone is not defined

Suddenly started to get this error when using vue-multiselect:
ReferenceError: deepClone is not defined
    at deepClone (eval at 167 (2.app.js:79), <anonymous>:8:20)
    at VueComponent.data (eval at 165 (2.app.js:65), <anonymous>:57:57)
    at VueComponent.mergedDataFn (eval at <anonymous> (app.js:146), <anonymous>:963:18)
    at VueComponent.mergedDataFn (eval at <anonymous> (app.js:146), <anonymous>:964:19)
    at initData (eval at <anonymous> (app.js:146), <anonymous>:3170:12)
    at initState (eval at <anonymous> (app.js:146), <anonymous>:3123:5)
    at VueComponent.Vue._init (eval at <anonymous> (app.js:146), <anonymous>:3376:5)
    at new VueComponent (eval at <anonymous> (app.js:146), <anonymous>:3498:12)
    at createComponentInstanceForVnode (eval at <anonymous> (app.js:146), <anonymous>:1728:10)
    at init (eval at <anonymous> (app.js:146), <anonymous>:1738:43)
logError @ vue.js?3de6:439

only in Chrome.
Did anyone face with it? Any ideas how to fix?

Pretty weird. How do you import the component? What’s your setup?

Yep, looks super strange. This is how I import the component:
import multiselect from 'vue-multiselect';
this is how I use it:
<multiselect v-model="value"
                     :options = "daysAll"
                     :multiple = "false"
                     label = "name"
                     select-label = ""
                     deselect-label = ""
                     selected-label = ""
                     placeholder="Day"
                     track-by = "val"
                     @input = "onInput">
        </multiselect>

I cleaned the browser cache, reinstalled npm dependencies, but still the same error.
I am using v2.0.0-beta.11 but it worked perfect before. Everything comes back to normal when I disable vue-multiselect. I see it only in Chrome on every page I have vue-multiselect. Do you know to what it relates to to point me to the right way?

Could you try this?
import Multiselect from 'vue-multiselect/lib/Multiselect'
Maybe there’s an error in the bundle. Also which version of Vue are you using?

When importing with:
 import Multiselect from 'vue-multiselect/lib/Multiselect'
there is an error:
Can't resolve 'vue-multiselect/lib/Multiselect' in...
My vue.js version is 2.1.10.

Super weird. Please wait until tomorrow/Sunday for a new version. Hopefully it will be fixed.

Thank you. Trying to find the issue by my own forces meanwhile. Will let you know if I get it fixed.

You might try updating to beta.13 and also updating vue-loader.

Breaking changes is the issue... but yes will have to if there will be no other way. Still trying to find the problem.

Any reason why you aren't using beta-13? It works for me.

@SirLamer the reason is braking changes. A lot of code to edit. But I will have to do that for sure.

Ok, trying to update to beta-13, but there is another issue while running gulp.
{ [Error: ./~/buble-loader!./~/vue-loader/lib/selector.js?type=script&index=0!./resources/assets/js/components/dashboard/work/jobs/list.vue
Module not found: Error: Can't resolve 'vue-multiselect' in '/home/vagrant/dev/project/resources/assets/js/components/dashboard/work/jobs'
resolve 'vue-multiselect' in '/home/vagrant/dev/project/resources/assets/js/components/dashboard/work/jobs'
  Parsed request is a module
....

I am not sure why it happens as it is not the issue with beta-11. It only happens with vue-multisilect beta-13. Trying to google the problem but with no results.
My package.json:
{
  "private": true,
  "scripts": {
    "prod": "gulp --production",
    "dev": "gulp watch"
  },
  "devDependencies": {
    "gulp": "^3.9.1",
    "laravel-echo": "^1.0.1",
    "laravel-elixir": "^6.0.0-14",
    "laravel-elixir-vue-2": "^0.2.0",
    "laravel-elixir-webpack-official": "^1.0.2",
    "lodash": "^4.16.6",
    "jquery": "^3.1.0",
    "es5-shim": "https://github.com/es-shims/es5-shim.git",
    "es6-shim": "https://github.com/es-shims/es6-shim.git",
    "es7-shim": "https://github.com/es-shims/es7-shim.git",
    "html5shiv": "^3.7.3",
    "bootstrap-sass": "^3.3.7",
    "csshake": "^1.5.1",
    "font-awesome": "^4.7.0",
    "moment": "^2.15",
    "pusher-js": "^3.2.1",
    "vue": "^2.1.0",
    "vue-multiselect": "v2.0.0-beta.13",
    "vue-resource": "^1.0.3",
    "vue-router": "^2.0.1",
    "vuex": "^2.0.0",
    "vee-validate": "2.0.0-beta.14",
    "vue-datepicker": "^1.3.0",
    "vue-infinite-scroll": "^2.0.0",
  },
  "dependencies": {

  }
}

Have you tried doing rm -rf node_modules? That sometimes helps. Also please make sure you use import Multiselect from 'vue-multiselect'. There might be a problem when trying to select something from /lib.

@shentao thank you. I found the way to fix the installation issue. I think some time before I had some problems with vue-multiselect that was fixed by an alias in webpack config:
module.exports = {

    output: {
        publicPath: "/assets/js/"
    },
    resolve: {
        alias: {
            'vue-multiselect': 'vue-multiselect/lib/Multiselect.vue'
        }

    }

};

So I removed that alias and was able to install beta-13. But still no success to fix the original problem of the thread:
ReferenceError: deepClone is not defined
    at deepClone (eval at 167 (2.app.js:79), <anonymous>:8:20)
    at VueComponent.data (eval at 165 (2.app.js:65), <anonymous>:57:57)
    at VueComponent.mergedDataFn (eval at <anonymous> (app.js:146), <anonymous>:963:18)
    at VueComponent.mergedDataFn (eval at <anonymous> (app.js:146), <anonymous>:964:19)
    at initData (eval at <anonymous> (app.js:146), <anonymous>:3170:12)
    at initState (eval at <anonymous> (app.js:146), <anonymous>:3123:5)
    at VueComponent.Vue._init (eval at <anonymous> (app.js:146), <anonymous>:3376:5)
    at new VueComponent (eval at <anonymous> (app.js:146), <anonymous>:3498:12)
    at createComponentInstanceForVnode (eval at <anonymous> (app.js:146), <anonymous>:1728:10)
    at init (eval at <anonymous> (app.js:146), <anonymous>:1738:43)

it happens even after the installation of beta-13 and only in Chrome :(

This should be fixed today/tomorrow! :D

@shentao that sounds great! do you know why does it happen? Is it because of the new vue.js version? Why only in Chrome? :O Is there any temporary fix before the beta.14 release?

I really have no idea, sorry :( Also, I use Chrome/Opera for development and I don’t get this message. It might be related to other libraries. Maybe vue-template-compiler? Vue itself or maybe even webpack. I guess you might try to make sure that you use the newest versions of everything.

For what it's worth, I work off of a forked branch of the project mostly because I want to build off of the .vue file and the vue-multiselect project team has decided to not distribute the source files with the project. So I fork it and edited .npmignore so that the /src folder is retrieved as well, then I include the Multiselect.vue. This compiles fine for me, also my javascript package ends up being a few kb smaller than otherwise when I include it in this way.

I guess I should add the /src files to the npm package then :) Also, most of the src files are available inside the /lib catalog.

@shentao hey, I managed to solve the issue with beta.13. Finally. Reinstalling all npm stuff and cleaning the cache helped. BTW, when are you going to push next release? There is an annoying bug: #187.

I work off of a branch of this project and can tell you that https://github.com/monterail/vue-multiselect/issues/187 is fixed in the 2.0 dev branch. I actually prepared to submit a PR to fix this then found it had already been done!

@SirLamer cool, thank you. And there are no breaking changes since beta.13, right?

Yeah I encountered this error for which I submitted a PR:
https://github.com/monterail/vue-multiselect/pull/264
It is still in the 2.0 branch here as the PR has not been accepted, but my fork has this PR already adopted.

@nikolaynesov @SirLamer beta.14 released. Sorry you had to wait. Please read: https://github.com/monterail/vue-multiselect/blob/2.0/README.md
Breaking change regarding the styles.

I think perhaps the wrong version of the package might have been published to NPM, and also there is no release under v1.1.4 on here.

@shentao thank you, but unfortunately I can not find a new beta version on GitHub. The most recent one is still v2.0.0-beta.13. Same issue as @SirLamer mentioned.

 Will be the first thing I will check in the morning! So sorry!
(A conference is not a good place to publish to npm :/)

Weird. Yesterday when I went to sleep I checked with unpkg if the new version is available and it wasn’t. But today morning it seems that everything is in place, so I guess it might have been a lag on npm’s side. :/

@shentao so the only breaking update between beta.13 and beta.14 is including styles separately:
<style src="vue-multiselect/dist/vue-multiselect.min.css"></style>
am I right?
is there any way to @import scss?

This and #274, can be considered breaking changes. The component uses just css so there are no scss files though there is nothing that should prevent you from importing the css into a scss file. It all depends on how your webpack setup looks like. Simple
@import vue-multiselect/dist/vue-multiselect.min inside the styles might work. :)

@shentao Tried to update to beta.14.
There is a new issue when settign v-model value to null.
[Vue warn]: Error in render function:
(found in <VueMultiselect>)
TypeError: Cannot read property 'val' of null
...

<multiselect v-model="value"
                     :options = "hours"
                     :multiple = "false"
                     :placeholder="placeholder"
                     select-label = ""
                     deselect-label = ""
                     selected-label = ""
                     track-by = "val"
                     label = "time"
                     @input = "onInput">
        </multiselect>

so rolled back to beta.13 and it came back to normal.

I can’t reproduce the error. :/ Could you provide a fiddle with your code?
Also, what for do you use the onInput method? Could you show me it’s code?

@shentao I created a custom component "time-input", that uses vue-multiselect and that I import to another component (parent). When I reset value in the parent component (I have a reset method that sets value to null), the "time-input" component gets it from the property and updates the value in "time-input". onInput method just emits updates from "time-input" component to parent component.
onInput() { this.$emit('updated', this.value) }

I don't think there is an attempt to read a property "val" anywhere inside vue-multiselect. Do you have an object property 'val' anywhere within your own component?

@SirLamer I think it happens because I have track-by = "val" in my code. First, I thought it is somewhere in my code, but after a review I found that I always check if the object value is not null before calling a val property on it. After I rolled back to beta.13 the error disappeared. I tried switching back and forward several times and having always the same result: beta.13 = ok. beta.14 = error. Will try doing the same once again tomorrow.

Weird. Yesterday I checked the case where you can init the component with null and also change an existing value to null. Everything works just fine. :/

@shentao I can init with null, but when I change it and set back to null there is an error.

Ohhhh okay, I think this is related to this PR: https://github.com/monterail/vue-multiselect/pull/264

@SirLamer what is the error you were getting in Chrome dev tools?

[Vue warn]: Error in render function:
(found in <VueMultiselect>)
TypeError: Cannot read property 'THE_VALUE_NAME_FROM_TRACK_BY_PROP' of null
...

Ok, I checked PR#264 and based on the fixes changed my component to check the variable type and if it is null then set value to [] an empty array. This is bit hacky but it's the only quick way. I really need to make a quick hotfix.
@SirLamer what do you think?

Yes that works because that's how the component is designed to nullify itself. I agree that it's hacky but it satisfies how vue-multiselect works. If my PR were adopted then providing "null" as input would work fine which is preferred.
This stems from one of the conditional checks on {whatever-the-selected-values-array-is-called}.length. It checks for a singular input and casts to an array but it does not do the same on a null input, so when the .length Array prototype method is called, it throws an exception.

Actually this only half fixes the problem, because if the user clears the input field then the component will emit "undefined" by default because it is looking for an element on an array which is empty. In my PR I changed it to return null if the array check returns a falsey result (such as 'undefined').
The PR changes are designed to minimize changes from the existing code, personally I would go about re-organizing how this stuff is tracked in the component but that's for another day, there are many PRs to consider and include, and doing a refactor beforehand would create a mess.

Guys chill. I will fix that tomorrow and release it immediately. I can see there is a PR for that already.

@SirLamer yep, I see... :( so the only way I guess it to fork 2.0 and apply your commit from #264?

@nikolaynesov I haven't tested, but I think you can work around the current build if you:

When you intend a clear/empty selection set, set 'value' on the component to an empty array, both for signular and multiple selection case.
When the 'input' event provides a value of 'undefined', then set your local variable value to null.

@SirLamer I guess it can also be done by watcher for value parameter... but it means editing every single component that uses vue-multiselect ... Hopefully @shentao will release new subversion with #264 included today...

You can also fork it and affect a fix within your forked copy.
I was asked to make a change on my PR, I am traveling for work but will try to affect it tonight.

@SirLamer I made a fork and affected the fix. Is there any way to publish the dist folder the same way as it is done in beta.14?

Yeah there is an NPM compile command, I forget what it is, check package.json.
Personally I edited the .npmignore so that the /src directory is distributed, and I include the source .vue file. Compile takes longer the result is smaller.


[ko] 3rd Korean localization work for v1.22

3rd Korean localization work for v1.22

/lgtm
/approve

/milestone dev-1.22-ko.3


Replace placeholder mod with provides

https://fabricmc.net/wiki/documentation:fabric_mod_json?s[]=provides

right, i was looking for that field, for some reasons i didn't find it


Just Testing: Fails on Interstitial BUT Banner and Video works!

I just copied the code while I can set Banner and Video (in testing mode) and it works like a charm,
But Interstitial fails as:
JS: Treating this deviceId as testdevice: 9627E477BBDDCCD2352E6BB85F8C052E
JS: onAdFailedToLoad: 3
JS: admob createInterstitial error: 3

and here is the code:
    admob.createInterstitial( {
        testing: true,
        androidInterstitialId: "", // add your own
        keywords: ["keyword1", "keyword2"], // add keywords for ad targeting
        onAdClosed: function () { console.log("interstitial closed") }
    } ).then(
        function() {
            console.log("admob createInterstitial done");
        },
        function(error) {
            console.log("admob createInterstitial error: " + error);
        }
    )

p.s. {N} v. 7

this works btw:
    admob.preloadRewardedVideoAd({
        testing: true,
        iosAdPlacementId: "ca-app-pub-XXXXXX/YYYYY2", // add your own
        androidAdPlacementId: "ca-app-pub-AAAAAAAA/BBBBBB2", // add your own
        keywords: ["keyword1", "keyword2"], // add keywords for ad targeting
    }).then(
        function() {

            admob.showRewardedVideoAd({
                onRewarded: (reward) => {
                    console.log("onRewarded");
                    this.message = "watched rewarded video";
                },
                onRewardedVideoAdLeftApplication: () => console.log("onRewardedVideoAdLeftApplication"),
                onRewardedVideoAdClosed: () => console.log("onRewardedVideoAdClosed"),
                onRewardedVideoAdOpened: () => console.log("onRewardedVideoAdOpened"),
                onRewardedVideoStarted: () => console.log("onRewardedVideoStarted"),
                onRewardedVideoCompleted: () => console.log("onRewardedVideoCompleted"),
            }).then(
                function() {
                    console.log("RewardedVideoAd showing");
                },
                function(error) {
                    console.log("admob showRewardedVideoAd error: " + error);
                }
            )

        },
        function(error) {
            console.log("admob preloadRewardedVideoAd error: " + error);
        }
    )

I can not figure it out why Interstitial fails, but any other methods just work as expected.

Did you ever figure it out? I can't even get a banner to show. The only thing I could get working was the reward video when using the preload reward video code like you have posted.


Feature - Moodle Integrated OAuth

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

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


Can't access video Stream

Hello,
I would like to access video stream to record video. But i can't find any method of JpegCamera class  to access the camera stream.
See here : https://github.com/amw/jpeg_camera/blob/6171a2915f3fe9eeab8e323e06391529282b2fce/src/jpeg_camera_html5.coffee#L87

JpegCamera does not support recording videos. It only takes static snapshots.

Ok,
Btw i figured out this by adding a stream parameter to JpegCamera object on line 89
that.stream = stream

@amw would not it be useful to give developers access to the stream?

I don't know. What are you using it for?
That stream object is only available in the HTML5 engine. It would brake the API when the flash fallback is used.


Can not change account after first account reach lv10?

Describe the bug
A clear and concise description of what the bug is.
Version
v2.2.0
Screenshots

Log Files
[07:47:13] [INFO   ] SUCCESS: Account Leveled
[07:47:13] [INFO   ] Terminating league related processes
[07:47:18] [INFO   ] Overwriting/creating game config
[07:47:18] [WARNING] No account set. Add accounts on account page
[07:47:19] [WARNING] League opened with prior login
[07:47:19] [INFO   ] Verifying logged-in account credentials
[07:47:22] [WARNING] Incorrect Account! Proceeding anyways
[07:47:26] [INFO   ] Checking for Client Updates
[07:47:26] [INFO   ] Client is up to date
[07:47:27] [ERROR  ] ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
[07:47:27] [ERROR  ] Traceback (most recent call last):
File "urllib3\connectionpool.py", line 790, in urlopen
File "urllib3\connectionpool.py", line 536, in _make_request
File "urllib3\connection.py", line 461, in getresponse
File "http\client.py", line 1411, in getresponse
File "http\client.py", line 324, in begin
File "http\client.py", line 293, in _read_status
http.client.RemoteDisconnected: Remote end closed connection without response
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "requests\adapters.py", line 486, in send
File "urllib3\connectionpool.py", line 844, in urlopen
File "urllib3\util\retry.py", line 470, in increment
File "urllib3\util\util.py", line 38, in reraise
File "urllib3\connectionpool.py", line 790, in urlopen
File "urllib3\connectionpool.py", line 536, in _make_request
File "urllib3\connection.py", line 461, in getresponse
File "http\client.py", line 1411, in getresponse
File "http\client.py", line 324, in begin
File "http\client.py", line 293, in _read_status
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "lolbot\bot\client.py", line 67, in account_loop
File "lolbot\bot\client.py", line 96, in leveling_loop
File "lolbot\bot\client.py", line 306, in account_leveled
File "lolbot\common\api.py", line 144, in request
File "requests\sessions.py", line 602, in get
File "requests\sessions.py", line 589, in request
File "requests\sessions.py", line 703, in send
File "requests\adapters.py", line 501, in send
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
[07:47:27] [ERROR  ] Unknown Error. Exiting
Additional context
Add any other context about the problem here.

04122023_1425.log

where is league of legends installed? this error seems odd, because if league is running there should always be a lockfile. Is your league of legends installed in C:/Riot Games/League of Legends  on your win 10 pc?

05122023_1032.log
Yes,C:\Riot Games\League of Legends.

Okay, i think this is a bug. Check your accounts tab and see if the 'level' actually matches up with the real level of the account. This if not, then reset the levels to 1 and close the league client and restart and it should work properly. This will be fixed in the next version (2.2.1)

add my DC id: xbabytank, I stream my game.

And it can't select new account 7 day champion reward when leve up.

Bot tab,restart UX work,close client not work.

This should be fixed in the latest bot version 2.3.0


As a User I want to reset my counter

Add a new button 'Reset' in manage-counter-menu to set amount of the counter to 0.


Export var not showing up in inspection

Godot version
4.02
System information
Macbook pro M1
Issue description
My exports dont show up in inspection. I made other exports on windows and those worked well.
It don´t work on eiter 4.02 or 4.01
@export var speed_multiplier: int = 1
@export var projectile: PackedScene
Steps to reproduce
@export var speed_multiplier: int = 1
@export var projectile: PackedScene
Minimal reproduction project
NA

It shomehow resolved itself.